Merge lp:~openstack-charmers/charms/precise/nova-cloud-controller/ceilometer-support into lp:~charmers/charms/precise/nova-cloud-controller/trunk

Proposed by James Page
Status: Merged
Merged at revision: 55
Proposed branch: lp:~openstack-charmers/charms/precise/nova-cloud-controller/ceilometer-support
Merge into: lp:~charmers/charms/precise/nova-cloud-controller/trunk
Diff against target: 41 lines (+10/-3)
2 files modified
config.yaml (+1/-1)
hooks/nova_cc_hooks.py (+9/-2)
To merge this branch: bzr merge lp:~openstack-charmers/charms/precise/nova-cloud-controller/ceilometer-support
Reviewer Review Type Date Requested Status
charmers Pending
Review via email: mp+194044@code.launchpad.net

Description of the change

Add support for use with ceilometer

Needs to be merged alongside merge proposal for quantum-gateway and nova-compute charms due to vhost change in messaging.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'config.yaml'
2--- config.yaml 2013-09-30 20:46:50 +0000
3+++ config.yaml 2013-11-06 00:55:57 +0000
4@@ -19,7 +19,7 @@
5 type: string
6 description: Username used to access rabbitmq queue
7 rabbit-vhost:
8- default: nova
9+ default: openstack
10 type: string
11 decsription: Rabbitmq vhost
12 database-user:
13
14=== modified file 'hooks/nova_cc_hooks.py'
15--- hooks/nova_cc_hooks.py 2013-10-23 16:55:26 +0000
16+++ hooks/nova_cc_hooks.py 2013-11-06 00:55:57 +0000
17@@ -103,8 +103,9 @@
18
19
20 @hooks.hook('amqp-relation-joined')
21-def amqp_joined():
22- relation_set(username=config('rabbit-user'), vhost=config('rabbit-vhost'))
23+def amqp_joined(relation_id=None):
24+ relation_set(relation_id=relation_id,
25+ username=config('rabbit-user'), vhost=config('rabbit-vhost'))
26
27
28 @hooks.hook('amqp-relation-changed')
29@@ -396,6 +397,12 @@
30 identity_joined(rid=rid)
31
32
33+@hooks.hook('upgrade-charm')
34+def upgrade_charm():
35+ for r_id in relation_ids('amqp'):
36+ amqp_joined(relation_id=r_id)
37+
38+
39 def main():
40 try:
41 hooks.execute(sys.argv)

Subscribers

People subscribed via source and target branches