Merge lp:~james-page/charms/precise/glance/notification into lp:~openstack-charmers-archive/charms/precise/glance/trunk

Proposed by James Page
Status: Merged
Merged at revision: 49
Proposed branch: lp:~james-page/charms/precise/glance/notification
Merge into: lp:~openstack-charmers-archive/charms/precise/glance/trunk
Diff against target: 30 lines (+7/-1)
2 files modified
templates/folsom/glance-api.conf (+3/-0)
templates/icehouse/glance-api.conf (+4/-1)
To merge this branch: bzr merge lp:~james-page/charms/precise/glance/notification
Reviewer Review Type Date Requested Status
Liam Young (community) Approve
OpenStack Charmers Pending
Review via email: mp+216573@code.launchpad.net

Description of the change

Fixup problems when glance is not related to rabbitmq-server

To post a comment you must log in.
Revision history for this message
James Page (james-page) wrote :

This can also be merged into the lp:charms/trusty/glance branch if OK

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

lgtm

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'templates/folsom/glance-api.conf'
2--- templates/folsom/glance-api.conf 2014-03-31 11:29:37 +0000
3+++ templates/folsom/glance-api.conf 2014-04-21 07:46:58 +0000
4@@ -30,7 +30,10 @@
5
6 {% include "parts/rabbitmq-single" %}
7
8+{% if rabbitmq_host or rabbitmq_hosts -%}
9 notifier_strategy = rabbit
10+{% endif -%}
11+
12 filesystem_store_datadir = /var/lib/glance/images/
13
14 {% if swift_store %}
15
16=== modified file 'templates/icehouse/glance-api.conf'
17--- templates/icehouse/glance-api.conf 2014-04-12 16:55:29 +0000
18+++ templates/icehouse/glance-api.conf 2014-04-21 07:46:58 +0000
19@@ -32,7 +32,10 @@
20
21 {% include "parts/rabbitmq" %}
22
23-notifier_strategy = rabbit
24+{% if rabbitmq_host or rabbitmq_hosts -%}
25+notification_driver = rabbit
26+{% endif -%}
27+
28 filesystem_store_datadir = /var/lib/glance/images/
29
30 {% if swift_store -%}

Subscribers

People subscribed via source and target branches