x
/
fuel-ccp-glance
Code Issues Proposed changes

Fix oslo notifications configs 

- enable glance-registry notifications ( searchlight )
- notification_driver is `noop` by default (means no notifications),
  so we can remove the `else` condition branch.
- this macros >
   {{ oslomessaging[messaging.backend.notifications]('notifications_config') }}
  should not depend on notifications consumer (searchlight,
  ceilometer, etc), it is just notifications transport backend-related
  config and must be commmon for all.

Change-Id: I16e9d604cb3d69e14d2124fb2ad4f603f86a87aa
This commit is contained in:
Kirill Bespalov 2016-12-29 13:19:44 +03:00
parent 6bc30fc525
commit 5e9f301e33
2 changed files with 12 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
service/files/glance-api.conf.j2
View File

@ -62,10 +62,9 @@ container_formats = bare
disk_formats = raw disk_formats = raw
{% endif %} {% endif %}
[oslo_messaging_notifications] {% if searchlight is defined and searchlight.services.glance -%}
{% if searchlight is defined and searchlight.services.glance %} [oslo_messaging_notifications]
driver = {{ searchlight.notification_driver }} driver = {{ searchlight.notification_driver }}
{{ oslo_messaging[messaging.backend.notifications]('notifications_config') }} {%- endif %}
{% else %}
driver = noop {{ oslomessaging[messaging.backend.notifications]('notifications_config') }}
{% endif %}

8
service/files/glance-registry.conf.j2
View File

@ -26,5 +26,9 @@ memcached_servers = {{ address('memcached', memcached.port) }}
[paste_deploy] [paste_deploy]
flavor = keystone flavor = keystone
[oslo_messaging_notifications] {% if searchlight is defined and searchlight.services.glance -%}
driver = noop [oslo_messaging_notifications]
driver = {{ searchlight.notification_driver }}
{%- endif %}
{{ oslomessaging[messaging.backend.notifications]('notifications_config') }}