Do not restart ovs when updating other_config:emc-insert-inv-prob

There are no requirement of service restart after updating
the cmd-insert-inv-prov parameter in ovs documentation[1], and
it seems that the parameter can be applied with keeping the service
running.
 [1] https://docs.openvswitch.org/en/latest/topics/dpdk/bridge/#emc-insertion-probability

Let's remove notify so that changing the value doesn't trigger
service restart.

Change-Id: I2e9f7d3b27547d33914f07005045b55d8ba82514
This commit is contained in:
Takashi Kajinami 2021-03-12 15:59:37 +09:00
parent 0d026451bd
commit 95230113b2
4 changed files with 6 additions and 8 deletions

View File

@ -120,9 +120,8 @@ class vswitch::dpdk (
if $disable_emc {
vs_config { 'other_config:emc-insert-inv-prob':
value => '0',
notify => Service['openvswitch'],
wait => false,
value => '0',
wait => false,
}
}

View File

@ -96,9 +96,8 @@ class vswitch::ovs(
if $disable_emc {
vs_config { 'other_config:emc-insert-inv-prob':
value => '0',
notify => Service['openvswitch'],
wait => false,
value => '0',
wait => false,
}
}

View File

@ -83,7 +83,7 @@ describe 'vswitch::dpdk' do
:value => 'true', :notify => 'Service[openvswitch]', :wait => true,
)
is_expected.to contain_vs_config('other_config:emc-insert-inv-prob').with(
:value => '0', :notify => 'Service[openvswitch]', :wait => false,
:value => '0', :wait => false,
)
is_expected.to contain_vs_config('other_config:vlan-limit').with(
:value => '2', :wait => true,

View File

@ -101,7 +101,7 @@ describe 'vswitch::ovs' do
end
it 'configures disable_emc option' do
is_expected.to contain_vs_config('other_config:emc-insert-inv-prob').with(
:value => '0', :notify => 'Service[openvswitch]', :wait => false,
:value => '0', :wait => false,
)
end
it 'configures vlan-limit option' do