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
1286518 log perms 2 #245
1286518 log perms 2 #245
Conversation
…g-perms-2 Conflicts: environs/cloudinit/cloudinit_test.go
LGTM although I dont really understand the full extent of the changes, mostly where exec is changed for script, I would like if someone else takes a look (also, if you think there is a slight possibility that this needs to be reverted by any issue, it would be a good idea to squash the changes into one to make it easy to look for in master. |
Thanks perrito666. A little background to help you feel more comfortable: in Upstart, apparently there can only be one exec line. A script clause is a way of executing >= 1 commands. |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
Relevant changes for use of Pebble in Juju: * fix: exec and checks inherit environment from daemon (#234) * Allow non-root to specify user/group to itself (#239) * Add support for services to configure working directory (#245) * Improve error messages with exec, especially if working dir missing (#236) * Add "service context" support for exec and health checks (#246) Full list of changes here: canonical/pebble@5842ea6...00bcd1f
Fix tutorial page
No description provided.