openstack/puppet-vswitch
Code Issues Proposed changes

Merge "Ensure other_config:emc-insert-inv-prob is unset"

Browse Source
This commit is contained in:
Zuul 2022-03-25 18:28:27 +00:00 committed by Gerrit Code Review
commit d57362f6cc
4 changed files with 16 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
manifests/dpdk.pp
View File

@ -141,6 +141,11 @@ class vswitch::dpdk (
value => 0,
wait => false,
}
} else {
vs_config { 'other_config:emc-insert-inv-prob':
ensure => absent,
wait => false,
}
}
if is_service_default($vlan_limit) {

5
manifests/ovs.pp
View File

@ -77,6 +77,11 @@ class vswitch::ovs(
value => 0,
wait => false,
}
} else {
vs_config { 'other_config:emc-insert-inv-prob':
ensure => absent,
wait => false,
}
}
if is_service_default($vlan_limit) {

4
spec/classes/vswitch_dpdk_spec.rb
View File

@ -53,7 +53,9 @@ describe 'vswitch::dpdk' do
is_expected.to contain_vs_config('other_config:hw-offload').with(
:ensure => 'absent', :restart => true, :wait => true,
)
is_expected.to_not contain_vs_config('other_config:emc-insert-inv-prob')
is_expected.to contain_vs_config('other_config:emc-insert-inv-prob').with(
:ensure => 'absent', :wait => false
)
is_expected.to contain_vs_config('other_config:vlan-limit').with(
:value => nil, :wait => true,
)

4
spec/classes/vswitch_ovs_spec.rb
View File

@ -28,7 +28,9 @@ describe 'vswitch::ovs' do
end
it 'configures disable_emc option to false' do
is_expected.to_not contain_vs_config('other_config:emc-insert-inv-prob')
is_expected.to contain_vs_config('other_config:emc-insert-inv-prob').with(
:ensure => 'absent', :wait => false
)
end
it 'clears vlan-limit option' do