On Thu, Jul 17, 2025 at 8:03 AM Ales Musil <amu...@redhat.com> wrote:
>
>
>
> On Thu, Jul 17, 2025 at 7:19 AM Han Zhou <hz...@ovn.org> wrote:
>>
>> The grep may accidentally show uuid that contains 4242. Fix it by grep
>> more accurately.
>>
>> Fixes: 137ed00a6604 ("ic: Advertise and learn routes for loadbalancer
VIPs.")
>> Signed-off-by: Han Zhou <hz...@ovn.org>
>> ---
>> 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 37d7bc1b6827..2221ae9bf7a2 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 4242::1 -A 2], [0], [dnl
>> ip_prefix : "4242::1/128"
>> nexthop : "2001:db8:1::1"
>> origin : loadbalancer
>> --
>> 2.38.1
>>
>> _______________________________________________
>> dev mailing list
>> d...@openvswitch.org
>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>>
>
> Hi Han,
>
> thank you for the patch, this has been addressed by Xavier in the longer
series already [0]. If you don't have any objections I would mark this one
as "Handled Elsewhere" and review/accept the fix as part of the larger
series.
>
> [0] https://patchwork.ozlabs.org/project/ovn/list/?series=464621
>
> Thanks,
> Ales
>
Oh, nice! Sorry that I didn't notice that one, and I encountered the
problem myself. Please ignore this patch.
Thanks,
Han
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
