c7b0cc82fa
We currently join the list into a comma-separated string. This does not make a lot of sense because oslo messaging notification drivers must not be passed in the driver=foo,bar form but in the driver=foo\ndriver=bar form. See: - https://docs.openstack.org/oslo.config/queens/configuration/format.html multi valued (MultiStrOpt) A multi-valued option is a string value and can be given more than once, all values will be used. # Driver or drivers to handle sending notifications. (multi valued) notification_driver = nova.openstack.common.notifier.rpc_notifier notification_driver = ceilometer.compute.nova_notifier By simply passing down the list we let all the puppet backends that use the openstack_config:ruby backend to work correctly (https://github.com/openstack/puppet-openstacklib#defined-provider-for-openstack_config-ruby) This will fix the scenario where multiple notification drivers are specified which is currently broken and will let the single driver scenario to keep working as is. Change-Id: If65946412b42e0919456ed92fdd8e3788ad67872 Related-Bug: #1851629 |
||
|---|---|---|
| .. | ||
| amqp.pp | ||
| default.pp | ||
| notifications.pp | ||
| rabbit.pp | ||
| zmq.pp | ||