openstack
/
openstack-ansible-os_tacker
Code Issues Proposed changes

Update messaging notification configuration 

This patch add the conditional inclusion of the notification
section of the service configuration. This ensures that oslo.messaging
notifications use the correct transport for deployments that have
separate rpc and notify messaging backends. For example, if the
transport_url is not provided in the notification section of the
service configuration, the transport_url specified in the default
section will be used instead.

This patch conditionally selects the notifier driver. The noop
driver will be selected when notification publishing is disabled.
The messagingv2 driver is selected when notification publishing is
enabled.

Change-Id: I60e92e14c0893b80f4023b6b6681864fee5228e5
Closes-Bug: #1794320
This commit is contained in:
ZhijunWei 2018-09-19 08:20:41 -04:00 committed by weizj
parent 1362769261
commit 56dc20030b
2 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
defaults/main.yml
View File

@ -20,6 +20,9 @@
debug: False
verbose: True
# Enable/Disable Ceilometer
tacker_ceilometer_enabled: False
# Set the host which will execute the shade modules
# for the service setup. The host must already have
# clouds.yaml properly configured.

1
templates/tacker.conf.j2
View File

@ -110,4 +110,5 @@ openstack = {{ tacker_etc_dir }}/vim/fernet_keys
ssl = {{ tacker_oslomsg_notify_use_ssl | bool }}
[oslo_messaging_notifications]
driver = {% if tacker_ceilometer_enabled %}messagingv2{% else %}noop{% endif %}
transport_url = {{ tacker_oslomsg_notify_transport }}://{% for host in tacker_oslomsg_notify_servers.split(',') %}{{ tacker_oslomsg_notify_userid }}:{{ tacker_oslomsg_notify_password }}@{{ host }}:{{ tacker_oslomsg_notify_port }}{% if not loop.last %},{% else %}/{{ tacker_oslomsg_notify_vhost }}{% if (tacker_oslomsg_notify_use_ssl | lower) | bool %}?ssl=1{% else %}?ssl=0{% endif %}{% endif %}{% endfor %}