Merge lp://staging/~vila/ubuntu-ci-services-itself/1290320-config-0600 into lp://staging/ubuntu-ci-services-itself
Proposed by
Vincent Ladeuil
| Status: | Merged |
|---|---|
| Approved by: | Vincent Ladeuil |
| Approved revision: | 346 |
| Merged at revision: | 351 |
| Proposed branch: | lp://staging/~vila/ubuntu-ci-services-itself/1290320-config-0600 |
| Merge into: | lp://staging/ubuntu-ci-services-itself |
| Diff against target: |
19 lines (+2/-0) 1 file modified
charms/precise/rabbitmq-worker/hooks/hooks.py (+2/-0) |
| To merge this branch: | bzr merge lp://staging/~vila/ubuntu-ci-services-itself/1290320-config-0600 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Andy Doan (community) | Approve | ||
| PS Jenkins bot (community) | continuous-integration | Needs Fixing | |
|
Review via email:
|
|||
Commit message
Fix chmod bits for temp files so they can be read by nobody/nogroup.
Description of the change
This fixes the chmod bits for the config file so it can be read by nobody/nogroup.
I can't explain why this wasn't detected earlier except if something else
changed (though, a change in tempfile.
Even with that fix, the workers can't be started anymore which reinforces my feeling that something has changed (see bug for details).
I'm proposing this fix anyway since that unbreaks trunk up to the point where others can participate in investigating the revealed bug (apparently we can't connect to the rabbit server socket anymore !??!!).
To post a comment you must log in.
Revision history for this message
| PS Jenkins bot (ps-jenkins) wrote | # |
review:
Needs Fixing
(continuous-integration)
Revision history for this message
| Andy Doan (doanac) | # |
review:
Approve
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
FAILED: Continuous integration, rev:346
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https:/
http://
Executed test runs:
Click here to trigger a rebuild:
http://