diff --git a/manifests/inspector.pp b/manifests/inspector.pp index feee751b..0715c0c5 100644 --- a/manifests/inspector.pp +++ b/manifests/inspector.pp @@ -25,6 +25,11 @@ # (optional) Define if the service must be enabled or not # Defaults to true # +# [*listen_address*] +# (optional) The listen IP for the Ironic-inspector API server. +# Should be an valid IP address +# Defaults to $::os_service_default. +# # [*pxe_transfer_protocol*] # (optional) Protocol to be used for transferring the ramdisk # Defaults to 'tftp'. Valid values are 'tftp' or 'http'. @@ -169,6 +174,7 @@ class ironic::inspector ( $package_ensure = 'present', $enabled = true, + $listen_address = $::os_service_default, $pxe_transfer_protocol = 'tftp', $enable_uefi = false, $debug = undef, @@ -284,6 +290,7 @@ class ironic::inspector ( } ironic_inspector_config { + 'DEFAULT/listen_address': value => $listen_address; 'DEFAULT/auth_strategy': value => $auth_strategy; 'firewall/dnsmasq_interface': value => $dnsmasq_interface; 'database/connection': value => $db_connection; diff --git a/spec/classes/ironic_inspector_spec.rb b/spec/classes/ironic_inspector_spec.rb index f7aca991..8b104d58 100644 --- a/spec/classes/ironic_inspector_spec.rb +++ b/spec/classes/ironic_inspector_spec.rb @@ -90,6 +90,7 @@ describe 'ironic::inspector' do end it 'configures inspector.conf' do + is_expected.to contain_ironic_inspector_config('DEFAULT/listen_address').with_value('') is_expected.to contain_ironic_inspector_config('DEFAULT/auth_strategy').with_value(p[:auth_strategy]) is_expected.to contain_ironic_inspector_config('keystone_authtoken/auth_type').with_value('password') is_expected.to contain_ironic_inspector_config('keystone_authtoken/auth_uri').with_value(p[:auth_uri]) @@ -151,6 +152,7 @@ describe 'ironic::inspector' do before :each do params.merge!( :debug => true, + :listen_address => '127.0.0.1', :auth_uri => 'http://192.168.0.1:5000/v2.0', :identity_uri => 'http://192.168.0.1:35357', :admin_password => 'password', @@ -166,6 +168,7 @@ describe 'ironic::inspector' do ) end it 'should replace default parameter with new value' do + is_expected.to contain_ironic_inspector_config('DEFAULT/listen_address').with_value(p[:listen_address]) is_expected.to contain_ironic_inspector_config('DEFAULT/debug').with_value(p[:debug]) is_expected.to contain_ironic_inspector_config('keystone_authtoken/auth_uri').with_value(p[:auth_uri]) is_expected.to contain_ironic_inspector_config('keystone_authtoken/auth_url').with_value(p[:identity_uri])