openstack
/
openstack
Code Issues Proposed changes

Update git submodules 

* Update puppet-oslo from branch 'master'
  - Merge "Messaging notifications should be set as a list"
  - Messaging notifications should be set as a list
    
    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
This commit is contained in:
Zuul 2020-04-14 06:45:55 +00:00 committed by Gerrit Code Review
parent 06b4b3837d
commit 4a4389d7a1
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
puppet-oslo

@ -1 +1 @@
Subproject commit 0560177096dba4731a3a97528a7a87fc69bb0f39
Subproject commit adc3147bcbe4509f254019faa62416e122b0b881