Skip to content

sudo instead of root#24

Open
arunvelsriram wants to merge 4 commits into
HashNuke:masterfrom
arunvelsriram:sudo_instead_of_root
Open

sudo instead of root#24
arunvelsriram wants to merge 4 commits into
HashNuke:masterfrom
arunvelsriram:sudo_instead_of_root

Conversation

@arunvelsriram

Copy link
Copy Markdown
  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

HashNuke commented Mar 6, 2016

Copy link
Copy Markdown
Owner

@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

HashNuke commented Mar 6, 2016

Copy link
Copy Markdown
Owner

@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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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