Notification drivers need to be a list
Convert the NotificationDriver to a comma_delimited_list. This will still not break existing templates because passing a string is still completely valid. This is done so that the hiera keys will be passed down as lists. The oslo::messaging::notifications::driver expects a list anyway so this won't break things and will allow us to actually specify multiple notification drivers correctly. The change that allowed oslo::notifications to use both strings and lists is If65946412b42e0919456ed92fdd8e3788ad67872 (Messaging notifications should be set as a list) Related-Bug: #1851629 Change-Id: I24c860cd3121e5c307233864818ca86967ff6d72
This commit is contained in:
parent
666091c949
commit
48d0e4d9b6
@ -58,7 +58,7 @@ parameters:
|
||||
default: 'regionOne'
|
||||
description: Keystone region for endpoint
|
||||
NotificationDriver:
|
||||
type: string
|
||||
type: comma_delimited_list
|
||||
default: 'noop'
|
||||
description: Driver or drivers to handle sending notifications.
|
||||
|
||||
|
@ -160,7 +160,7 @@ parameters:
|
||||
default: {}
|
||||
type: json
|
||||
NotificationDriver:
|
||||
type: string
|
||||
type: comma_delimited_list
|
||||
default: 'noop'
|
||||
description: Driver or drivers to handle sending notifications.
|
||||
DeployIdentifier:
|
||||
|
@ -73,7 +73,7 @@ parameters:
|
||||
type: string
|
||||
hidden: true
|
||||
NotificationDriver:
|
||||
type: string
|
||||
type: comma_delimited_list
|
||||
default: 'noop'
|
||||
description: Driver or drivers to handle sending notifications.
|
||||
GnocchiArchivePolicy:
|
||||
|
@ -81,7 +81,7 @@ parameters:
|
||||
default: {}
|
||||
type: json
|
||||
NotificationDriver:
|
||||
type: string
|
||||
type: comma_delimited_list
|
||||
default: 'noop'
|
||||
description: Driver or drivers to handle sending notifications.
|
||||
RootStackName:
|
||||
|
@ -51,7 +51,7 @@ parameters:
|
||||
type: string
|
||||
hidden: true
|
||||
NotificationDriver:
|
||||
type: string
|
||||
type: comma_delimited_list
|
||||
default: 'noop'
|
||||
description: Driver or drivers to handle sending notifications.
|
||||
KeystoneRegion:
|
||||
|
@ -63,7 +63,7 @@ parameters:
|
||||
description: Set rabbit subscriber port, change this if using SSL
|
||||
type: number
|
||||
NotificationDriver:
|
||||
type: string
|
||||
type: comma_delimited_list
|
||||
default: 'noop'
|
||||
description: Driver or drivers to handle sending notifications.
|
||||
|
||||
|
@ -209,7 +209,7 @@ parameters:
|
||||
default: {}
|
||||
type: json
|
||||
NotificationDriver:
|
||||
type: string
|
||||
type: comma_delimited_list
|
||||
default: 'noop'
|
||||
description: Driver or drivers to handle sending notifications.
|
||||
EnableInternalTLS:
|
||||
|
@ -124,7 +124,7 @@ parameters:
|
||||
description: Maximum raw byte size of the Heat API JSON request body.
|
||||
type: number
|
||||
NotificationDriver:
|
||||
type: string
|
||||
type: comma_delimited_list
|
||||
default: 'noop'
|
||||
description: Driver or drivers to handle sending notifications.
|
||||
HeatCorsAllowedOrigin:
|
||||
|
@ -175,7 +175,7 @@ parameters:
|
||||
default: {}
|
||||
hidden: true
|
||||
NotificationDriver:
|
||||
type: string
|
||||
type: comma_delimited_list
|
||||
default: 'noop'
|
||||
description: Driver or drivers to handle sending notifications.
|
||||
KeystoneChangePasswordUponFirstUse:
|
||||
|
@ -54,7 +54,7 @@ parameters:
|
||||
type: string
|
||||
hidden: true
|
||||
NotificationDriver:
|
||||
type: string
|
||||
type: comma_delimited_list
|
||||
default: 'noop'
|
||||
description: Driver or drivers to handle sending notifications.
|
||||
|
||||
|
@ -102,7 +102,7 @@ parameters:
|
||||
via parameter_defaults in the resource registry.
|
||||
type: json
|
||||
NotificationDriver:
|
||||
type: string
|
||||
type: comma_delimited_list
|
||||
default: 'noop'
|
||||
description: Driver or drivers to handle sending notifications.
|
||||
EnableVLANTransparency:
|
||||
|
@ -35,7 +35,7 @@ parameters:
|
||||
default: 'regionOne'
|
||||
description: Keystone region for endpoint
|
||||
NotificationDriver:
|
||||
type: string
|
||||
type: comma_delimited_list
|
||||
default: 'noop'
|
||||
description: Driver or drivers to handle sending notifications.
|
||||
NovaPassword:
|
||||
|
@ -52,7 +52,7 @@ parameters:
|
||||
Remove configuration that is not generated by TripleO. Used to avoid
|
||||
configuration remnants after upgrades.
|
||||
NotificationDriver:
|
||||
type: string
|
||||
type: comma_delimited_list
|
||||
default: 'noop'
|
||||
description: Driver or drivers to handle sending notifications.
|
||||
OctaviaUserName:
|
||||
|
Loading…
Reference in New Issue
Block a user