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.

Sign up for GitHub

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

Jump to bottom

Set cwd for install commands in salt state file #51

Open
taylor-peterson opened this issue Jun 5, 2016 · 0 comments
Open

Set cwd for install commands in salt state file #51

taylor-peterson opened this issue Jun 5, 2016 · 0 comments

Comments

@taylor-peterson
Copy link
Owner

taylor-peterson commented Jun 5, 2016 •
edited
Loading

Volnoti and rofi install scripts currently set the cwd before running because saltstack/salt#11497 currently isn't sorted out for 2016.03.0.

When that's fixed, the "cd /tmp/*" commands should be removed from /install.sh and replaced with "- cwd: /tmp/" in the appropriate *.sls files.
taylor-peterson pushed a commit that referenced this issue Jun 5, 2016
Temporary order of operations fix
952fa59 
cmd.run currently runs cwd before requisites. This means that if the
cwd directory won't exist if the unless suceeds, the state will fail.
For now, we remove the cwd command and replace it with a cd in the
install script. A proper fix is tracked by #51.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@taylor-peterson