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

change webhook exec to execute the command built previously #70

Merged
merged 1 commit into from
Aug 26, 2014

Conversation

k-f
Copy link

@k-f k-f commented Aug 24, 2014

We've had a couple of issues with the mcollective agent. As a short term workaround I tried changing the webhook to run r10k directly but it doesn't work without this change

acidprime added a commit that referenced this pull request Aug 26, 2014
change webhook exec to execute the command built previously
@acidprime acidprime merged commit 245ffbf into voxpupuli:master Aug 26, 2014
@acidprime
Copy link
Collaborator

Note that this will require you to run the hook as root, which I have params for but have not tested recently

@acidprime
Copy link
Collaborator

@acidprime
Copy link
Collaborator

@k-f any luck getting this working as root?

@k-f
Copy link
Author

k-f commented Sep 15, 2014

Yep, it's all running fine. Cheers

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