On 2/13/26 3:23 PM, Rukomoinikova Aleksandra wrote:
> On 13.02.2026 16:50, Alexandra Rukomoinikova wrote:
>> Fixes: 7b0eb4d9ed05 ("northd: Add distributed load balancer support.")
>> Signed-off-by: Alexandra Rukomoinikova <[email protected]>
>> ---
>> northd/northd.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/northd/northd.c b/northd/northd.c
>> index 4ecdd9c2d..0749aa049 100644
>> --- a/northd/northd.c
>> +++ b/northd/northd.c
>> @@ -12899,6 +12899,7 @@ build_distr_lrouter_nat_flows_for_lb(struct
>> lrouter_nat_lb_flows_ctx *ctx,
>> * the undnat stage.
>> */
>> ds_put_format(ctx->undnat_match, ") && outport == %s", dgp->json_key);
>> + ds_clear(ctx->gw_redir_action);
>> ds_put_format(ctx->gw_redir_action,
>> "outport = %s; next;", gw_outport);
>>
>> --
>> 2.48.1
>>
> Hi Dumitru! I tested this fix here
> https://github.com/Sashhkaa/ovn/actions/runs/21988153806 — just in case
> it's easier to check.
>
Hi Alexandra,
Thanks a lot for the quick fix, it makes sense! I chuckled a bit when I
saw how your branch was called. :)
To have a bit more context on why this was needed, I added this to the
commit log:
Reported-at:
https://mail.openvswitch.org/pipermail/ovs-dev/2026-February/430261.html
I'd normally ask for a flow test in ovn-northd.at to avoid this
happening in the future. But because our CI is perma-failing, I'll
accept the patch as is now.
Would you have time to update one of the existing ovn-northd.at tests to
cover this flow too as a follow up at some point (not urgent)?
Applied to main.
Regards,
Dumitru
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
