[ovs-dev] [PATCH ovn 2/4] system-ovn.at: Use ADD_BR macro instead of bare ovs-vsctl.

Mark Gray mark.d.gray at redhat.com
Fri Jun 4 14:50:52 UTC 2021


On 03/06/2021 16:05, Dumitru Ceara wrote:
> This ensures the datapath type is properly set for the bridge.

I don't understand how this causes a problem? Can you provide additional
context?

> 
> a8362ff85dee ("northd: Fix the missing force_snat_for_lb flows when router_ip is configured.")
> Signed-off-by: Dumitru Ceara <dceara at redhat.com>
> ---
>  tests/system-ovn.at |    3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/tests/system-ovn.at b/tests/system-ovn.at
> index 76fc21c87..310bd3d5a 100644
> --- a/tests/system-ovn.at
> +++ b/tests/system-ovn.at
> @@ -2312,8 +2312,7 @@ CHECK_CONNTRACK_NAT()
>  ovn_start
>  OVS_TRAFFIC_VSWITCHD_START()
>  ADD_BR([br-int])
> -check ovs-vsctl add-br br-ext
> -
> +ADD_BR([br-ext], [set Bridge br-ext fail-mode=standalone])
>  
>  # Set external-ids in br-int needed for ovn-controller
>  ovs-vsctl \
> 
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> 



More information about the dev mailing list