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

sudo instead of root #24

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

sudo instead of root #24

wants to merge 4 commits into from

Conversation

arunvelsriram
Copy link

  1. Since running applications as root is not recommended I extracted remote_user as a variable so that the default value set for it is not root.
  2. I used sudo for commands that require escalated privileges. The password for sudo can be managed by modifying the sudoers file or by using ansible-vault.
  3. Used ~ to find the correct home directory of deployer.
  4. Specify the roles path with -p option provided by ansible-galaxy to have the roles inside the project directory itself.

@HashNuke
Copy link
Owner

HashNuke commented Mar 6, 2016

@arunvelsriram Will need time to look into this. There's some parallel refactoring work going on in the upgrade-fixes branch. https://github.com/HashNuke/ansible-elixir-stack/tree/upgrade-fixes

@HashNuke
Copy link
Owner

HashNuke commented Mar 6, 2016

@arunvelsriram I would rather have contributions based on the upgrade-fixes branch than the master since there'll most likely be a lot of conflicts. Here's a list of pending issues on that branch #23

@arunvelsriram
Copy link
Author

@HashNuke Yes its quite a lot of changes. Please take your own time.
I have already started working on the upgrade-fixes branch. Will make a pull request soon.

@arunvelsriram
Copy link
Author

Hi. I checked the upgrade-fixes branch. Looks like refactoring is going well. May I know if the code on upgrade-fixes branch is stable and ready to use ?

@HashNuke
Copy link
Owner

@arunvelsriram Not yet. Almost there. I tried out distillery and have notes. Need to make changes.

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