Skip to content

Add deploy to digital ocean button#66

Draft
cfu288 wants to merge 9 commits intofastenhealth:mainfrom
cfu288:add-deploy-to-digital-ocean-button
Draft

Add deploy to digital ocean button#66
cfu288 wants to merge 9 commits intofastenhealth:mainfrom
cfu288:add-deploy-to-digital-ocean-button

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

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

</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.

2 participants