Merge lp://qastaging/~1chb1n/charms/trusty/rabbitmq-server/lp1485722-pidfile into lp://qastaging/~openstack-charmers-archive/charms/trusty/rabbitmq-server/next
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 106 | ||||
Proposed branch: | lp://qastaging/~1chb1n/charms/trusty/rabbitmq-server/lp1485722-pidfile | ||||
Merge into: | lp://qastaging/~openstack-charmers-archive/charms/trusty/rabbitmq-server/next | ||||
Diff against target: |
16 lines (+4/-1) 1 file modified
scripts/collect_rabbitmq_stats.sh (+4/-1) |
||||
To merge this branch: | bzr merge lp://qastaging/~1chb1n/charms/trusty/rabbitmq-server/lp1485722-pidfile | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Billy Olsen | Approve | ||
Review via email: mp+269288@code.qastaging.launchpad.net |
Description of the change
Address rabbitmq pidfile location differences for Vivid. bug 1485722
Amulet tests in this branch are irrelevant as they only exercise Trusty-Icehouse, they do not exercise nrpe, and they do not accurately test clustering success. There are proposed tests in a pending proposal which also contains the pidfile fixes proposed here. I will link that amulet test result as an indicator that this resolves the issue with Vivid, and preserves existing behavior below Vivid.
I recommend landing this pidfile fix ahead of the amulet-refactor fix.
FYI here is the amulet-refactor branch:
https:/
charm_lint_check #8813 rabbitmq- server- next for 1chb1n mp269288
LINT OK: passed
Build: http:// 10.245. 162.77: 8080/job/ charm_lint_ check/8813/