Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add deploy to digital ocean button #66

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

cfu288
Copy link

@cfu288 cfu288 commented Feb 12, 2023

This PR adds a '1-click deploy to Digital Ocean' button to the README and adds a Digital Ocean app template to specify the ENV variables to use for the deploy.

Quick video of the flow. Note that DO by default does not pick their cheap $5 instance, but instead a $12 instance. You have to manually downgrade the instance.

Screen.Recording.2023-02-11.at.7.44.17.PM.mov

I need to remove the --debug I added in from the Dockerfile and update the links from my fork to this repo before merging. The hard links are necessary for now in case anyone wants to test it, but the links should be updated to point at the main branch after testing.

@CLAassistant
Copy link

CLAassistant commented Feb 12, 2023

CLA assistant check
All committers have signed the CLA.

- dockerfile_path: Dockerfile
name: app
git:
branch: add-deploy-to-digital-ocean-button
Copy link
Author

@cfu288 cfu288 Feb 12, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

branch needs to be updated to main before merge

name: app
git:
branch: add-deploy-to-digital-ocean-button
repo_clone_url: https://github.com/cfu288/fasten-onprem.git
Copy link
Author

@cfu288 cfu288 Feb 12, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

repo_clone_url needs to be updated to https://github.com/fastenhealth/fasten-onprem.git before merge

@@ -51,7 +51,7 @@ COPY --from=backend-build /go/bin/fasten /opt/fasten/fasten
COPY LICENSE.md /opt/fasten/LICENSE.md
COPY config.yaml /opt/fasten/config/config.yaml

CMD ["/opt/fasten/fasten", "start", "--config", "/opt/fasten/config/config.yaml"]
CMD ["/opt/fasten/fasten", "start", "--config", "/opt/fasten/config/config.yaml", "--debug"]
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

--debug needs to be removed before merge

@@ -4,6 +4,12 @@
</a>
</p>

<p align="center">
<a href="https://cloud.digitalocean.com/apps/new?repo=https://github.com/cfu288/fasten-onprem/tree/add-deploy-to-digital-ocean-button">
Copy link
Author

@cfu288 cfu288 Feb 12, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

href needs to be updated to https://cloud.digitalocean.com/apps/new?repo=https://github.com/fastenhealth/fasten-onprem/tree/main before deploy

@cfu288
Copy link
Author

cfu288 commented Feb 12, 2023

Screenshot 2023-02-11 at 8 13 03 PM

Running with --debug shows that the secret is being successfully replaced.

Before this is merged, I need to update all the hardlinks from my fork to the main repo. I've just left them pointing to my fork for now in case you want to test out the button before merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants