On 7/1/26 12:04 PM, Xavier Simonart via dev wrote:
> Test was sending packets right after changing ACL, w/o guaranteeing
> that flows were installed, but expected that ACL was working as
> expected.
> 
> Signed-off-by: Xavier Simonart <[email protected]>
> ---
>  tests/system-ovn.at | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/system-ovn.at b/tests/system-ovn.at
> index ed5d63fd3..871b1f586 100644
> --- a/tests/system-ovn.at
> +++ b/tests/system-ovn.at
> @@ -14361,7 +14361,7 @@ test
>  acl_uuid=$(fetch_column nb:ACL _uuid priority=1000)
>  
>  # Update the ACL so that it no longer matches our client-server traffic
> -check ovn-nbctl set ACL $acl_uuid match="\"ip4.dst == 192.168.1.3\""
> +check ovn-nbctl --wait=hv set ACL $acl_uuid match="\"ip4.dst == 
> 192.168.1.3\""
>  
>  # Send another packet from the client to the server.
>  AT_CHECK([printf "test\n" > /tmp/myfifo], [0], [dnl

Hi Xavier,

Thanks for the fix, applied to main and 26.03.

Regards,
Dumitru

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to