openstack/puppet-ironic
Code Issues Proposed changes

Expose the notification_level paramenter

Browse Source 
After merging https://review.openstack.org/#/c/581786/
oslo notifications are unusable in ironic unless the
notification_level parameter is set. Thus, there is a
need to expose this parameter via puppet

Change-Id: I737501a010e310354c36d7b879132c9fefbff86e
This commit is contained in:
Daniel Abad 2018-08-30 13:58:05 +02:00
parent 7f0e56d305
commit 27bf3a0591
3 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
manifests/init.pp
View File

@ -266,6 +266,10 @@
# (optional) AMQP topic used for OpenStack notifications
# Defaults to $::os_service_default
#
# [*notification_level*]
# (optional) Notification level for outgoing notifications
# Defaults to $::os_service_default
#
class ironic (
$enabled = true,
$package_ensure = 'present',
@ -322,6 +326,7 @@ class ironic (
$notification_transport_url = $::os_service_default,
$notification_driver = $::os_service_default,
$notification_topics = $::os_service_default,
$notification_level = $::os_service_default,
) {
include ::ironic::deps
@ -351,6 +356,7 @@ class ironic (
'DEFAULT/auth_strategy': value => $auth_strategy;
'DEFAULT/my_ip': value => $my_ip;
'DEFAULT/default_resource_class': value => $default_resource_class;
'DEFAULT/notification_level': value => $notification_level;
}
if $sync_db {

4
releasenotes/notes/add-notification-level-7d21ada84df44ca5.yaml Normal file
View File

@ -0,0 +1,4 @@
---
features:
- |
Expose the ``notification_level`` parameter in ``::ironic``

1
spec/classes/ironic_init_spec.rb
View File

@ -113,6 +113,7 @@ describe 'ironic' do
is_expected.to contain_ironic_config('DEFAULT/control_exchange').with_value('<SERVICE DEFAULT>')
is_expected.to contain_ironic_config('DEFAULT/transport_url').with_value('<SERVICE DEFAULT>').with_secret(true)
is_expected.to contain_ironic_config('DEFAULT/default_resource_class').with_value('<SERVICE DEFAULT>')
is_expected.to contain_ironic_config('DEFAULT/notification_level').with_value('<SERVICE DEFAULT>')
end
end