Merge lp:~negronjl/charms/precise/rabbitmq-server/lp1312291 into lp:charms/rabbitmq-server

Proposed by Juan L. Negron
Status: Merged
Merged at revision: 53
Proposed branch: lp:~negronjl/charms/precise/rabbitmq-server/lp1312291
Merge into: lp:charms/rabbitmq-server
Diff against target: 25 lines (+6/-2)
1 file modified
hooks/rabbitmq_server_relations.py (+6/-2)
To merge this branch: bzr merge lp:~negronjl/charms/precise/rabbitmq-server/lp1312291
Reviewer Review Type Date Requested Status
Marco Ceppi (community) Approve
Review via email: mp+217087@code.launchpad.net

Description of the change

Fix for lp:1312291

To post a comment you must log in.
Revision history for this message
Marco Ceppi (marcoceppi) wrote :

LGTM, +1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'hooks/rabbitmq_server_relations.py'
2--- hooks/rabbitmq_server_relations.py 2014-03-28 11:32:22 +0000
3+++ hooks/rabbitmq_server_relations.py 2014-04-24 16:33:37 +0000
4@@ -56,6 +56,8 @@
5 SERVICE_NAME = os.getenv('JUJU_UNIT_NAME').split('/')[0]
6 POOL_NAME = SERVICE_NAME
7 RABBIT_DIR = '/var/lib/rabbitmq'
8+RABBIT_USER = 'rabbitmq'
9+RABBIT_GROUP = 'rabbitmq'
10 NAGIOS_PLUGINS = '/usr/local/lib/nagios/plugins'
11
12
13@@ -331,8 +333,10 @@
14 rbd_img=rbd_img, sizemb=sizemb,
15 fstype='ext4', mount_point=RABBIT_DIR,
16 blk_device=blk_device,
17- system_services=['rabbitmq-server'],
18- rbd_pool_replicas=rbd_pool_rep_count)
19+ system_services=['rabbitmq-server'])#,
20+ #rbd_pool_replicas=rbd_pool_rep_count)
21+ subprocess.check_call(['chown', '-R', '%s:%s' %
22+ (RABBIT_USER,RABBIT_GROUP), RABBIT_DIR])
23 else:
24 log('This is not the peer leader. Not configuring RBD.')
25 log('Stopping rabbitmq-server.')

Subscribers

People subscribed via source and target branches