Bug 2162632

Summary: [RHOSP 17.1][Aodh] Multivalued options created with wrong syntax
Product: Red Hat OpenStack Reporter: Eric Nothen <enothen>
Component: puppet-aodhAssignee: Yadnesh Kulkarni <ykulkarn>
Status: CLOSED ERRATA QA Contact: Leonid Natapov <lnatapov>
Severity: medium Docs Contact: Michael Geary <mgeary>
Priority: medium    
Version: 17.1 (Wallaby)CC: jjoyce, joflynn, jschluet, mrunge, slinaber, tkajinam, tvignaud, ykulkarn
Target Milestone: betaKeywords: Triaged, ZStream
Target Release: 17.1Flags: ykulkarn: needinfo+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: puppet-aodh-18.4.2-1.20230131221616.3e47b5a.el9ost Doc Type: Bug Fix
Doc Text:
Before this update, values of multi-value parameters were not populated correctly in the Alarming service (aodh) configuration because input to multi-value parameters was not considered as an array instead as a single value. With this update, you can set multiple values for a parameter and all values are populated in a configuration file.
Story Points: ---
Clone Of: Environment:
Last Closed: 2023-08-16 01:13:18 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Eric Nothen 2023-01-20 09:27:33 UTC
This bug was initially created as a copy of Bug #2042429

I am copying this bug because: 

The issue was resolved for:

- nova
- neutron
- keystone

But the same issue is still present in other services, such as:

- glance
- cinder
- octavia
- manila
- ceilometer
- heat
- designate
- barbican
- aodh
- ironic


Description of problem:
The notification_driver is a multivalued parameter, i.e. several drivers can be enabled, but the current code only enables the first string.

Given this code:
~~~
[stack.lab ~]$ cat hieradata.yaml 
aodh::notification_driver:
  - "log"
  - "messagingv2"
[stack.lab ~]$ 
~~~

The following is implemented:
~~~
[stack.lab ~]$ sudo egrep "^\[oslo_messaging_notifications|^driver" /var/lib/config-data/puppet-generated/aodh/etc/aodh/aodh.conf
[oslo_messaging_notifications]
driver=log
[stack.lab ~]$ 
~~~

It should actually look like this:
~~~
[oslo_messaging_notifications]
driver=log
driver=messagingv2
~~~

Comment 8 Leonid Natapov 2023-05-07 15:44:26 UTC
Fiexed.

[root@controller-0 tripleo-admin]# sudo egrep "^\[oslo_messaging_notifications|^driver" /var/lib/config-data/puppet-generated/aodh/etc/aodh/aodh.conf
[oslo_messaging_notifications]
driver=messagingv2

Comment 18 errata-xmlrpc 2023-08-16 01:13:18 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Release of components for Red Hat OpenStack Platform 17.1 (Wallaby)), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHEA-2023:4577

Comment 19 Red Hat Bugzilla 2023-12-15 04:25:56 UTC
The needinfo request[s] on this closed bug have been removed as they have been unresolved for 120 days