openstack/puppet-neutron
Code Issues Proposed changes

Support [ovn] broadcast_arps_to_all_routers

Browse Source 
Depends-on: https://review.opendev.org/923391
Change-Id: Ic810db84ede30d730aea998e8ced75ea96164d78
This commit is contained in:
Takashi Kajinami 2024-08-15 10:36:02 +09:00
parent 04c8fdd06f
commit 84690bdf17
3 changed files with 17 additions and 0 deletions

8
manifests/plugins/ml2/ovn.pp

@ -136,6 +136,12 @@
# (optional) The number of seconds to keep MAC_Binding entries in the OVN DB. # (optional) The number of seconds to keep MAC_Binding entries in the OVN DB.
# Defaults to $facts['os_service_default'] # Defaults to $facts['os_service_default']
# #
# [*broadcast_arps_to_all_routers*]
# (optional) If enabled OVM will flood ARP requests to all attached ports on
# a network. Supported by OVM >= 23.06.
# Type: boolean
# Defaults to $facts['os_service_default']
#
# [*ignore_lsp_down*] # [*ignore_lsp_down*]
# (optional) Do not install ARP/ND reply flows for logical switch ports # (optional) Do not install ARP/ND reply flows for logical switch ports
# if the port is DOWN. # if the port is DOWN.
@ -192,6 +198,7 @@ class neutron::plugins::ml2::ovn(
$localnet_learn_fdb = $facts['os_service_default'], $localnet_learn_fdb = $facts['os_service_default'],
$fdb_age_threshold = $facts['os_service_default'], $fdb_age_threshold = $facts['os_service_default'],
$mac_binding_age_threshold = $facts['os_service_default'], $mac_binding_age_threshold = $facts['os_service_default'],
$broadcast_arps_to_all_routers = $facts['os_service_default'],
$ignore_lsp_down = $facts['os_service_default'], $ignore_lsp_down = $facts['os_service_default'],
$fdb_removal_limit = $facts['os_service_default'], $fdb_removal_limit = $facts['os_service_default'],
$mac_binding_removal_limit = $facts['os_service_default'], $mac_binding_removal_limit = $facts['os_service_default'],
@ -241,6 +248,7 @@ class neutron::plugins::ml2::ovn(
'ovn/localnet_learn_fdb' : value => $localnet_learn_fdb; 'ovn/localnet_learn_fdb' : value => $localnet_learn_fdb;
'ovn/fdb_age_threshold' : value => $fdb_age_threshold; 'ovn/fdb_age_threshold' : value => $fdb_age_threshold;
'ovn/mac_binding_age_threshold' : value => $mac_binding_age_threshold; 'ovn/mac_binding_age_threshold' : value => $mac_binding_age_threshold;
'ovn/broadcast_arps_to_all_routers' : value => $broadcast_arps_to_all_routers;
'ovn_nb_global/ignore_lsp_down' : value => $ignore_lsp_down; 'ovn_nb_global/ignore_lsp_down' : value => $ignore_lsp_down;
'ovn_nb_global/fdb_removal_limit' : value => $fdb_removal_limit; 'ovn_nb_global/fdb_removal_limit' : value => $fdb_removal_limit;
'ovn_nb_global/mac_binding_removal_limit' : value => $mac_binding_removal_limit; 'ovn_nb_global/mac_binding_removal_limit' : value => $mac_binding_removal_limit;

5
releasenotes/notes/broadcast_arps_to_all_routers-e2535180f8ee5808.yaml Normal file

@ -0,0 +1,5 @@
---
features:
- |
The ``neutron::plugins::ml2::ovn::broadcast_arps_to_all_routers`` parameter
has been added.

4
spec/classes/neutron_plugins_ml2_ovn_spec.rb

@ -35,6 +35,7 @@ describe 'neutron::plugins::ml2::ovn' do
:localnet_learn_fdb => '<SERVICE DEFAULT>', :localnet_learn_fdb => '<SERVICE DEFAULT>',
:fdb_age_threshold => '<SERVICE DEFAULT>', :fdb_age_threshold => '<SERVICE DEFAULT>',
:mac_binding_age_threshold => '<SERVICE DEFAULT>', :mac_binding_age_threshold => '<SERVICE DEFAULT>',
:broadcast_arps_to_all_routers => '<SERVICE DEFAULT>',
:ignore_lsp_down => '<SERVICE DEFAULT>', :ignore_lsp_down => '<SERVICE DEFAULT>',
:fdb_removal_limit => '<SERVICE DEFAULT>', :fdb_removal_limit => '<SERVICE DEFAULT>',
:mac_binding_removal_limit => '<SERVICE DEFAULT>', :mac_binding_removal_limit => '<SERVICE DEFAULT>',
@ -80,6 +81,7 @@ describe 'neutron::plugins::ml2::ovn' do
should contain_neutron_plugin_ml2('ovn/localnet_learn_fdb').with_value(p[:localnet_learn_fdb]) should contain_neutron_plugin_ml2('ovn/localnet_learn_fdb').with_value(p[:localnet_learn_fdb])
should contain_neutron_plugin_ml2('ovn/fdb_age_threshold').with_value(p[:fdb_age_threshold]) should contain_neutron_plugin_ml2('ovn/fdb_age_threshold').with_value(p[:fdb_age_threshold])
should contain_neutron_plugin_ml2('ovn/mac_binding_age_threshold').with_value(p[:mac_binding_age_threshold]) should contain_neutron_plugin_ml2('ovn/mac_binding_age_threshold').with_value(p[:mac_binding_age_threshold])
should contain_neutron_plugin_ml2('ovn/broadcast_arps_to_all_routers').with_value(p[:broadcast_arps_to_all_routers])
should contain_neutron_plugin_ml2('ovn_nb_global/ignore_lsp_down').with_value(p[:ignore_lsp_down]) should contain_neutron_plugin_ml2('ovn_nb_global/ignore_lsp_down').with_value(p[:ignore_lsp_down])
should contain_neutron_plugin_ml2('ovn_nb_global/fdb_removal_limit').with_value(p[:fdb_removal_limit]) should contain_neutron_plugin_ml2('ovn_nb_global/fdb_removal_limit').with_value(p[:fdb_removal_limit])
should contain_neutron_plugin_ml2('ovn_nb_global/mac_binding_removal_limit').with_value(p[:mac_binding_removal_limit]) should contain_neutron_plugin_ml2('ovn_nb_global/mac_binding_removal_limit').with_value(p[:mac_binding_removal_limit])
@ -116,6 +118,7 @@ describe 'neutron::plugins::ml2::ovn' do
:localnet_learn_fdb => false, :localnet_learn_fdb => false,
:fdb_age_threshold => 10, :fdb_age_threshold => 10,
:mac_binding_age_threshold => 11, :mac_binding_age_threshold => 11,
:broadcast_arps_to_all_routers => true,
:ignore_lsp_down => false, :ignore_lsp_down => false,
:fdb_removal_limit => 12, :fdb_removal_limit => 12,
:mac_binding_removal_limit => 13, :mac_binding_removal_limit => 13,
@ -147,6 +150,7 @@ describe 'neutron::plugins::ml2::ovn' do
should contain_neutron_plugin_ml2('ovn/localnet_learn_fdb').with_value(p[:localnet_learn_fdb]) should contain_neutron_plugin_ml2('ovn/localnet_learn_fdb').with_value(p[:localnet_learn_fdb])
should contain_neutron_plugin_ml2('ovn/fdb_age_threshold').with_value(p[:fdb_age_threshold]) should contain_neutron_plugin_ml2('ovn/fdb_age_threshold').with_value(p[:fdb_age_threshold])
should contain_neutron_plugin_ml2('ovn/mac_binding_age_threshold').with_value(p[:mac_binding_age_threshold]) should contain_neutron_plugin_ml2('ovn/mac_binding_age_threshold').with_value(p[:mac_binding_age_threshold])
should contain_neutron_plugin_ml2('ovn/broadcast_arps_to_all_routers').with_value(p[:broadcast_arps_to_all_routers])
should contain_neutron_plugin_ml2('ovn_nb_global/ignore_lsp_down').with_value(p[:ignore_lsp_down]) should contain_neutron_plugin_ml2('ovn_nb_global/ignore_lsp_down').with_value(p[:ignore_lsp_down])
should contain_neutron_plugin_ml2('ovn_nb_global/fdb_removal_limit').with_value(p[:fdb_removal_limit]) should contain_neutron_plugin_ml2('ovn_nb_global/fdb_removal_limit').with_value(p[:fdb_removal_limit])
should contain_neutron_plugin_ml2('ovn_nb_global/mac_binding_removal_limit').with_value(p[:mac_binding_removal_limit]) should contain_neutron_plugin_ml2('ovn_nb_global/mac_binding_removal_limit').with_value(p[:mac_binding_removal_limit])