It is impossible to set multiple drivers for oslo messaging notifications

Bug #1851629 reported by Michele Baldessari
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Incomplete
High
Michele Baldessari

Bug Description

Currently in tripleo we have a parameter called NotificationDriver which is defined as follows:
  NotificationDriver:
    type: string
    default: 'messagingv2'
    description: Driver or drivers to handle sending notifications.

The problem is that it is impossible to set multiple driver notifications which is desired from time to time. Note that oslo messaging expects multiple drivers to be specified in separate and multiple 'driver=foo\ndriver=bar' lines and *not* in a single 'driver=foo,bar' line [1].

The reason this is not possible is many-fold:
A) Most puppet modules (aodh, cinder, glance, heat, keystone) use the traditional ini puppet backend that does not allow for multiple lines of the same key within the same section.

B) puppet-oslo defines the oslo::messaging::notifications::driver parameter as list but then it actually converts it into a comma separated string:
https://github.com/openstack/puppet-oslo/blob/master/manifests/messaging/notifications.pp#L10

C) Tripleo Heat templates are always passing the notification_driver hiera keys as strings

[1] From https://docs.openstack.org/oslo.config/queens/configuration/format.html we have the following:

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

Revision history for this message
Michele Baldessari (michele) wrote :
Changed in tripleo:
milestone: ussuri-1 → ussuri-2
wes hayutin (weshayutin)
Changed in tripleo:
milestone: ussuri-2 → ussuri-3
wes hayutin (weshayutin)
Changed in tripleo:
milestone: ussuri-3 → ussuri-rc3
wes hayutin (weshayutin)
Changed in tripleo:
milestone: ussuri-rc3 → victoria-1
Changed in tripleo:
milestone: victoria-1 → victoria-3
Changed in tripleo:
milestone: victoria-3 → wallaby-1
Changed in tripleo:
milestone: wallaby-1 → wallaby-2
Changed in tripleo:
milestone: wallaby-2 → wallaby-3
Revision history for this message
Marios Andreou (marios-b) wrote :

This is an automated action. Bug status has been set to 'Incomplete' and target milestone has been removed due to inactivity. If you disagree please re-set these values and reach out to us on freenode #tripleo

Changed in tripleo:
milestone: wallaby-3 → none
status: Triaged → Incomplete
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.