Bug 2162632 - [RHOSP 17.1][Aodh] Multivalued options created with wrong syntax
Summary: [RHOSP 17.1][Aodh] Multivalued options created with wrong syntax
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat OpenStack
Classification: Red Hat
Component: puppet-aodh
Version: 17.1 (Wallaby)
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: beta
: 17.1
Assignee: Yadnesh Kulkarni
QA Contact: Leonid Natapov
Michael Geary
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2023-01-20 09:27 UTC by Eric Nothen
Modified: 2023-12-15 04:25 UTC (History)
8 users (show)

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.
Clone Of:
Environment:
Last Closed: 2023-08-16 01:13:18 UTC
Target Upstream Version:
Embargoed:
ykulkarn: needinfo+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
OpenStack gerrit 800598 0 None MERGED Add support for MultiStrOpt 2023-01-23 01:39:05 UTC
OpenStack gerrit 871504 0 None MERGED Add support for MultiStrOpt 2023-01-31 10:06:05 UTC
Red Hat Issue Tracker OSP-21594 0 None None None 2023-01-20 09:29:46 UTC
Red Hat Product Errata RHEA-2023:4577 0 None None None 2023-08-16 01:13:41 UTC

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


Note You need to log in before you can comment on or make changes to this bug.