openstack/puppet-neutron
Code Issues Proposed changes

Add support for [ovn] ovn_router_indirect_snat

Browse Source 
... which was added during this cycle.

Depends-on: https://review.opendev.org/926495
Change-Id: Ib53d335714283a719dcb99013d1bad224386a1ae
This commit is contained in:
Takashi Kajinami 2024-09-10 10:21:06 +09:00
parent a4d9425fe8
commit 4110754021
3 changed files with 16 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
manifests/plugins/ml2/ovn.pp
View File

@ -142,6 +142,11 @@
# Type: boolean
# Defaults to $facts['os_service_default']
#
# [*ovn_router_indirect_snat*]
# (optional) Whether to configure SNAT for all nested subnets connected to
# the router through any routers.
# Defaults to $facts['os_service_default']
#
# [*ignore_lsp_down*]
# (optional) Do not install ARP/ND reply flows for logical switch ports
# if the port is DOWN.
@ -199,6 +204,7 @@ class neutron::plugins::ml2::ovn(
$fdb_age_threshold = $facts['os_service_default'],
$mac_binding_age_threshold = $facts['os_service_default'],
$broadcast_arps_to_all_routers = $facts['os_service_default'],
$ovn_router_indirect_snat = $facts['os_service_default'],
$ignore_lsp_down = $facts['os_service_default'],
$fdb_removal_limit = $facts['os_service_default'],
$mac_binding_removal_limit = $facts['os_service_default'],
@ -249,6 +255,7 @@ class neutron::plugins::ml2::ovn(
'ovn/fdb_age_threshold' : value => $fdb_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/ovn_router_indirect_snat' : value => $ovn_router_indirect_snat;
'ovn_nb_global/ignore_lsp_down' : value => $ignore_lsp_down;
'ovn_nb_global/fdb_removal_limit' : value => $fdb_removal_limit;
'ovn_nb_global/mac_binding_removal_limit' : value => $mac_binding_removal_limit;

5
releasenotes/notes/ovn_router_indirect_snat-735d59868ddc9ed2.yaml Normal file
View File

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

4
spec/classes/neutron_plugins_ml2_ovn_spec.rb
View File

@ -36,6 +36,7 @@ describe 'neutron::plugins::ml2::ovn' do
:fdb_age_threshold => '<SERVICE DEFAULT>',
:mac_binding_age_threshold => '<SERVICE DEFAULT>',
:broadcast_arps_to_all_routers => '<SERVICE DEFAULT>',
:ovn_router_indirect_snat => '<SERVICE DEFAULT>',
:ignore_lsp_down => '<SERVICE DEFAULT>',
:fdb_removal_limit => '<SERVICE DEFAULT>',
:mac_binding_removal_limit => '<SERVICE DEFAULT>',
@ -82,6 +83,7 @@ describe 'neutron::plugins::ml2::ovn' do
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/broadcast_arps_to_all_routers').with_value(p[:broadcast_arps_to_all_routers])
should contain_neutron_plugin_ml2('ovn/ovn_router_indirect_snat').with_value(p[:ovn_router_indirect_snat])
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/mac_binding_removal_limit').with_value(p[:mac_binding_removal_limit])
@ -119,6 +121,7 @@ describe 'neutron::plugins::ml2::ovn' do
:fdb_age_threshold => 10,
:mac_binding_age_threshold => 11,
:broadcast_arps_to_all_routers => true,
:ovn_router_indirect_snat => false,
:ignore_lsp_down => false,
:fdb_removal_limit => 12,
:mac_binding_removal_limit => 13,
@ -151,6 +154,7 @@ describe 'neutron::plugins::ml2::ovn' do
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/broadcast_arps_to_all_routers').with_value(p[:broadcast_arps_to_all_routers])
should contain_neutron_plugin_ml2('ovn/ovn_router_indirect_snat').with_value(p[:ovn_router_indirect_snat])
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/mac_binding_removal_limit').with_value(p[:mac_binding_removal_limit])