diff --git a/manifests/neutron.pp b/manifests/neutron.pp index fcec5044..f49870b0 100644 --- a/manifests/neutron.pp +++ b/manifests/neutron.pp @@ -61,6 +61,10 @@ # successfully lease addresses. # Defaults to $facts['os_service_default'] # +# [*fail_on_port_binding_failure*] +# (Optional) Fail deployment if neutron port binding fails. +# Defaults to $facts['os_service_default'] +# class ironic::neutron ( $password, $auth_type = 'password', @@ -73,6 +77,7 @@ class ironic::neutron ( $region_name = $facts['os_service_default'], $endpoint_override = $facts['os_service_default'], $dhcpv6_stateful_address_count = $facts['os_service_default'], + $fail_on_port_binding_failure = $facts['os_service_default'], ) { if is_service_default($system_scope) { @@ -95,5 +100,6 @@ class ironic::neutron ( 'neutron/region_name': value => $region_name; 'neutron/endpoint_override': value => $endpoint_override; 'neutron/dhcpv6_stateful_address_count': value => $dhcpv6_stateful_address_count; + 'neutron/fail_on_port_binding_failure': value => $fail_on_port_binding_failure; } } diff --git a/releasenotes/notes/neutron-fail_on_port_binding_failure-e55b4311d8a09f69.yaml b/releasenotes/notes/neutron-fail_on_port_binding_failure-e55b4311d8a09f69.yaml new file mode 100644 index 00000000..ff7f057b --- /dev/null +++ b/releasenotes/notes/neutron-fail_on_port_binding_failure-e55b4311d8a09f69.yaml @@ -0,0 +1,5 @@ +--- +features: + - | + The new ``ironic::neutron::fail_on_port_binding_failure`` parameter has + been added. diff --git a/spec/classes/ironic_neutron_spec.rb b/spec/classes/ironic_neutron_spec.rb index b62a08b6..180e4ea6 100644 --- a/spec/classes/ironic_neutron_spec.rb +++ b/spec/classes/ironic_neutron_spec.rb @@ -34,6 +34,7 @@ describe 'ironic::neutron' do is_expected.to contain_ironic_config('neutron/region_name').with_value('') is_expected.to contain_ironic_config('neutron/endpoint_override').with_value('') is_expected.to contain_ironic_config('neutron/dhcpv6_stateful_address_count').with_value('') + is_expected.to contain_ironic_config('neutron/fail_on_port_binding_failure').with_value('') end context 'when overriding parameters' do @@ -48,6 +49,7 @@ describe 'ironic::neutron' do :region_name => 'regionTwo', :endpoint_override => 'http://example2.com', :dhcpv6_stateful_address_count => 8, + :fail_on_port_binding_failure => true, ) end @@ -62,6 +64,7 @@ describe 'ironic::neutron' do is_expected.to contain_ironic_config('neutron/region_name').with_value(params[:region_name]) is_expected.to contain_ironic_config('neutron/endpoint_override').with_value(params[:endpoint_override]) is_expected.to contain_ironic_config('neutron/dhcpv6_stateful_address_count').with_value(params[:dhcpv6_stateful_address_count]) + is_expected.to contain_ironic_config('neutron/fail_on_port_binding_failure').with_value(true) end end