Add OVS ovsdb_timeout parameter

This adds the ovsdb_timeout to the ovs config
section which is the seconds that it should try
ovsdb commands before failing. [1]

[1] https://github.com/openstack/neutron/blob/master/neutron/conf/agent/ovs_conf.py#L24

Change-Id: Id73e8ae40f4db67247bde36156e3b84c456b9f29
(cherry picked from commit fbafd406fc)
(cherry picked from commit 235c69cbc8)
This commit is contained in:
Tobias Urdin 2020-02-05 11:00:58 +01:00
parent 470cd27ef6
commit 6c3b600d6c
3 changed files with 21 additions and 0 deletions

View File

@ -42,6 +42,10 @@
# (optional) List of <physical_network>:<bridge> # (optional) List of <physical_network>:<bridge>
# Defaults to empty list # Defaults to empty list
# #
# [*ovsdb_timeout*]
# (Optional) The timeout in seconds for OVSDB commands.
# Defaults to $::os_service_default
#
# [*integration_bridge*] # [*integration_bridge*]
# (optional) Integration bridge in OVS # (optional) Integration bridge in OVS
# Defaults to 'br-int' # Defaults to 'br-int'
@ -167,6 +171,7 @@ class neutron::agents::ml2::ovs (
$extensions = $::os_service_default, $extensions = $::os_service_default,
$bridge_uplinks = [], $bridge_uplinks = [],
$bridge_mappings = [], $bridge_mappings = [],
$ovsdb_timeout = $::os_service_default,
$integration_bridge = 'br-int', $integration_bridge = 'br-int',
$tunnel_types = [], $tunnel_types = [],
$local_ip = false, $local_ip = false,
@ -291,6 +296,7 @@ class neutron::agents::ml2::ovs (
'agent/extensions': value => join(any2array($extensions), ','); 'agent/extensions': value => join(any2array($extensions), ',');
'agent/minimize_polling': value => $minimize_polling; 'agent/minimize_polling': value => $minimize_polling;
'agent/tunnel_csum': value => $tunnel_csum; 'agent/tunnel_csum': value => $tunnel_csum;
'ovs/ovsdb_timeout': value => $ovsdb_timeout;
'ovs/integration_bridge': value => $integration_bridge; 'ovs/integration_bridge': value => $integration_bridge;
'ovs/datapath_type': value => $datapath_type; 'ovs/datapath_type': value => $datapath_type;
'ovs/vhostuser_socket_dir': value => $vhostuser_socket_dir; 'ovs/vhostuser_socket_dir': value => $vhostuser_socket_dir;

View File

@ -0,0 +1,4 @@
---
features:
- |
Added new parameter neutron::agents::ml2::ovs::ovsdb_timeout.

View File

@ -50,6 +50,7 @@ describe 'neutron::agents::ml2::ovs' do
should contain_neutron_agent_ovs('ovs/datapath_type').with_value(['<SERVICE DEFAULT>']) should contain_neutron_agent_ovs('ovs/datapath_type').with_value(['<SERVICE DEFAULT>'])
should contain_neutron_agent_ovs('ovs/vhostuser_socket_dir').with_value(['<SERVICE DEFAULT>']) should contain_neutron_agent_ovs('ovs/vhostuser_socket_dir').with_value(['<SERVICE DEFAULT>'])
should contain_neutron_agent_ovs('ovs/ovsdb_interface').with_value(['<SERVICE DEFAULT>']) should contain_neutron_agent_ovs('ovs/ovsdb_interface').with_value(['<SERVICE DEFAULT>'])
should contain_neutron_agent_ovs('ovs/ovsdb_timeout').with_value('<SERVICE DEFAULT>')
should contain_neutron_agent_ovs('ovs/integration_bridge').with_value(p[:integration_bridge]) should contain_neutron_agent_ovs('ovs/integration_bridge').with_value(p[:integration_bridge])
should contain_neutron_agent_ovs('securitygroup/firewall_driver').\ should contain_neutron_agent_ovs('securitygroup/firewall_driver').\
with_value(p[:firewall_driver]) with_value(p[:firewall_driver])
@ -193,6 +194,16 @@ describe 'neutron::agents::ml2::ovs' do
end end
end end
context 'when setting ovsdb_timeout' do
before :each do
params.merge!( :ovsdb_timeout => 30 )
end
it 'configures ovsdb_timeout' do
should contain_neutron_agent_ovs('ovs/ovsdb_timeout').with_value(params[:ovsdb_timeout])
end
end
context 'when supplying extensions for ML2 plugin' do context 'when supplying extensions for ML2 plugin' do
before :each do before :each do
params.merge!(:extensions => ['qos']) params.merge!(:extensions => ['qos'])