Merge lp:~thedac/charms/trusty/glance/optional-amqp into lp:~openstack-charmers-archive/charms/trusty/glance/next

Proposed by David Ames
Status: Merged
Merged at revision: 148
Proposed branch: lp:~thedac/charms/trusty/glance/optional-amqp
Merge into: lp:~openstack-charmers-archive/charms/trusty/glance/next
Diff against target: 21 lines (+3/-1)
1 file modified
hooks/glance_utils.py (+3/-1)
To merge this branch: bzr merge lp:~thedac/charms/trusty/glance/optional-amqp
Reviewer Review Type Date Requested Status
Liam Young (community) Approve
Review via email: mp+273542@code.launchpad.net

Description of the change

Make messaging an optional relation for workload status

To post a comment you must log in.
Revision history for this message
Liam Young (gnuoy) wrote :

Approved

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

charm_lint_check #11388 glance-next for thedac mp273542
    LINT OK: passed

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

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

charm_unit_test #10580 glance-next for thedac mp273542
    UNIT OK: passed

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

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

charm_amulet_test #7135 glance-next for thedac mp273542
    AMULET OK: passed

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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'hooks/glance_utils.py'
2--- hooks/glance_utils.py 2015-10-02 21:27:28 +0000
3+++ hooks/glance_utils.py 2015-10-06 15:01:23 +0000
4@@ -125,7 +125,6 @@
5 # list has a complete context.
6 REQUIRED_INTERFACES = {
7 'database': ['shared-db', 'pgsql-db'],
8- 'messaging': ['amqp'],
9 'identity': ['identity-service'],
10 }
11
12@@ -453,6 +452,9 @@
13 if relation_ids('ceph') or relation_ids('object-store'):
14 required_interfaces['storage-backend'] = ['ceph', 'object-store']
15
16+ if relation_ids('amqp'):
17+ required_interfaces['messaging'] = ['amqp']
18+
19 if required_interfaces:
20 set_os_workload_status(configs, required_interfaces)
21 return status_get()

Subscribers

People subscribed via source and target branches