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

Setup capistrano for deployments #228

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Setup capistrano for deployments #228

wants to merge 1 commit into from

Conversation

bagedevimo
Copy link
Contributor

The current deployment process doesn't give us many options for
rollbacks, and could have issues if the server was restarted during the
deployment. Most rails apps that aren't containerised tend to be
deployed with Capistrano [1].

Capistrano lets us all the same user, and manage deployment access by
giving people access to the deploy user by adding their ssh public key.
We should be able to move towards that being a sudo-less enterprise, but
qless may make that difficult.

The current deployment process doesn't give us many options for
rollbacks, and could have issues if the server was restarted during the
deployment. Most rails apps that aren't containerised tend to be
deployed with Capistrano [1].

Capistrano lets us all the same user, and manage deployment access by
giving people access to the deploy user by adding their ssh public key.
We should be able to move towards that being a sudo-less enterprise, but
qless may make that difficult.
Copy link
Contributor Author

bagedevimo commented Dec 29, 2023

This stack of pull requests is managed by Graphite. Learn more about stacking.

Comment on lines +6 to +7
# server "train.nzoi.org.nz", user: "cap_deploy", roles: %w[app db web]
server "faketrain", user: "cap_deploy", roles: %w[app db web]
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this just from debugging?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah, this needs some decisions - we could use a new user, or make some changes to the existing deploy user, and i wanted to make sure that if this got accidentally merged it couldn't do any damage.

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