diff mbox series

[ovs-dev,10/11] tests: Fix flaky "ovn-ic -- route sync -- loadbalancer".

Message ID 20250710153345.104954-11-xsimonar@redhat.com
State Superseded
Headers show
Series Multiple unit/system tests fixes. | expand

Checks

Context Check Description
ovsrobot/apply-robot success apply and check: success
ovsrobot/github-robot-_Build_and_Test success github build: passed
ovsrobot/github-robot-_ovn-kubernetes fail github build: failed

Commit Message

Xavier Simonart July 10, 2025, 3:33 p.m. UTC
The test was sometimes failing as matching parts of uuids.

Signed-off-by: Xavier Simonart <xsimonar@redhat.com>
---
 tests/ovn-ic.at | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Mark Michelson July 21, 2025, 8:49 p.m. UTC | #1
Acked-by: Mark Michelson <mmichels@redhat.com>

On 7/10/25 11:33 AM, Xavier Simonart via dev wrote:
> The test was sometimes failing as matching parts of uuids.
> 
> Signed-off-by: Xavier Simonart <xsimonar@redhat.com>
> ---
>   tests/ovn-ic.at | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/ovn-ic.at b/tests/ovn-ic.at
> index 37d7bc1b6..30220c12e 100644
> --- a/tests/ovn-ic.at
> +++ b/tests/ovn-ic.at
> @@ -912,7 +912,7 @@ check ovn_as az1 ovn-nbctl lb-add lb_v6 [[4242::1]]:80 "[[4242::2]]:80"
>   check ovn_as az1 ovn-nbctl lr-lb-add lr1 lb_v6
>   OVS_WAIT_UNTIL([ovn_as az2 ovn-nbctl lr-route-list lr2 | grep learned | grep 4242])
>   
> -AT_CHECK([ovn-ic-sbctl list route | grep 4242 -A 2], [0], [dnl
> +AT_CHECK([ovn-ic-sbctl list route | grep 'ip_prefix.*4242' -A 2], [0], [dnl
>   ip_prefix           : "4242::1/128"
>   nexthop             : "2001:db8:1::1"
>   origin              : loadbalancer
diff mbox series

Patch

diff --git a/tests/ovn-ic.at b/tests/ovn-ic.at
index 37d7bc1b6..30220c12e 100644
--- a/tests/ovn-ic.at
+++ b/tests/ovn-ic.at
@@ -912,7 +912,7 @@  check ovn_as az1 ovn-nbctl lb-add lb_v6 [[4242::1]]:80 "[[4242::2]]:80"
 check ovn_as az1 ovn-nbctl lr-lb-add lr1 lb_v6
 OVS_WAIT_UNTIL([ovn_as az2 ovn-nbctl lr-route-list lr2 | grep learned | grep 4242])
 
-AT_CHECK([ovn-ic-sbctl list route | grep 4242 -A 2], [0], [dnl
+AT_CHECK([ovn-ic-sbctl list route | grep 'ip_prefix.*4242' -A 2], [0], [dnl
 ip_prefix           : "4242::1/128"
 nexthop             : "2001:db8:1::1"
 origin              : loadbalancer