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

chown after rsync upload in deployment #24

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

MiroRadenovic
Copy link
Contributor

the problem is that rsync uploads and creates folders :/srv/apps/ec2/v/0.0.9 with the root user(sudo) and the application runs with the ubuntu user.
this is not a good solution if your application needs to create temporary data in the project's folder
with this fix a chown -R ubuntu:ubuntu is run after the upload

let me know what do you think,
thx

@bevacqua
Copy link
Owner

Sounds fine but ubuntu:ubuntu should use conf('AWS_SSH_USER') instead of hardcoding ubuntu

@MiroRadenovic
Copy link
Contributor Author

ok, i will fix that

@MiroRadenovic
Copy link
Contributor Author

done.
as soon as you merge it, i will PL for #23
thank you

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