openstack
/
puppet-ironic
Code Issues Proposed changes

Merge "Expose the notification_level paramenter"

This commit is contained in:
Zuul 2018-09-02 21:49:34 +00:00 committed by Gerrit Code Review
parent 4519c6f887 27bf3a0591
commit 624c7135cb
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

@ -117,6 +117,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