Merge lp:~jjo/charms/trusty/swift-storage/fix-upstart-nrpe-checks into lp:~openstack-charmers-archive/charms/trusty/swift-storage/next

Proposed by JuanJo Ciarlante
Status: Merged
Merged at revision: 66
Proposed branch: lp:~jjo/charms/trusty/swift-storage/fix-upstart-nrpe-checks
Merge into: lp:~openstack-charmers-archive/charms/trusty/swift-storage/next
Diff against target: 36 lines (+13/-13)
1 file modified
hooks/swift_storage_utils.py (+13/-13)
To merge this branch: bzr merge lp:~jjo/charms/trusty/swift-storage/fix-upstart-nrpe-checks
Reviewer Review Type Date Requested Status
Liam Young (community) Approve
Mick Gregg (community) Approve
Review via email: mp+263597@code.launchpad.net
To post a comment you must log in.
Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_lint_check #5711 swift-storage-next for jjo mp263597
    LINT OK: passed

Build: http://10.245.162.77:8080/job/charm_lint_check/5711/

Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_unit_test #5343 swift-storage-next for jjo mp263597
    UNIT OK: passed

Build: http://10.245.162.77:8080/job/charm_unit_test/5343/

Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_amulet_test #4903 swift-storage-next for jjo mp263597
    AMULET OK: passed

Build: http://10.245.162.77:8080/job/charm_amulet_test/4903/

Revision history for this message
Mick Gregg (macgreagoir) wrote :

Confirmed service names and use in update_nrpe_config(). LGTM

review: Approve
Revision history for this message
Liam Young (gnuoy) wrote :

Approve

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'hooks/swift_storage_utils.py'
2--- hooks/swift_storage_utils.py 2015-04-01 16:05:59 +0000
3+++ hooks/swift_storage_utils.py 2015-07-01 21:38:03 +0000
4@@ -84,19 +84,19 @@
5 ]
6
7 SWIFT_SVCS = [
8- 'account-auditor',
9- 'account-reaper',
10- 'account-replicator',
11- 'account-server',
12- 'container-auditor',
13- 'container-replicator',
14- 'container-server',
15- 'container-sync',
16- 'container-updater',
17- 'object-auditor',
18- 'object-replicator',
19- 'object-server',
20- 'object-updater',
21+ 'swift-account-auditor',
22+ 'swift-account-reaper',
23+ 'swift-account-replicator',
24+ 'swift-account-server',
25+ 'swift-container-auditor',
26+ 'swift-container-replicator',
27+ 'swift-container-server',
28+ 'swift-container-sync',
29+ 'swift-container-updater',
30+ 'swift-object-auditor',
31+ 'swift-object-replicator',
32+ 'swift-object-server',
33+ 'swift-object-updater',
34 ]
35
36 RESTART_MAP = {

Subscribers

People subscribed via source and target branches