On Mon, Mar 21, 2022 at 9:50 AM Numan Siddique <[email protected]> wrote:
>
> On Mon, Mar 21, 2022 at 12:26 PM Numan Siddique <[email protected]> wrote:
> >
> > On Mon, Mar 21, 2022 at 12:21 PM Numan Siddique <[email protected]> wrote:
> > >
> > > On Sun, Mar 13, 2022 at 8:01 PM Han Zhou <[email protected]> wrote:
> > > >
> > > > Add a new action ct_lb_mark, which is the same as ct_lb except that
it
> > > > internally uses ct_mark to store the NAT flag, while ct_lb uses
ct_label
> > > > for the same purpose. This will be used later to move the masked
access
> > > > of ct_label to ct_mark while keeping the backward compatibility.
> > > >
> > > > Signed-off-by: Han Zhou <[email protected]>
> > >
> > > Acked-by: Numan Siddique <[email protected]>
> >
> > Looks like there is a system test failure in the CI run.  Can you
> > please check if its because of this patch or its a flaky one.
> >
> >
> > 97: Load balancer health checks -- ovn-northd -- dp-groups=yes FAILED
> > (ovs-macros.at:255)
> >  98: Load balancer health checks -- ovn-northd -- dp-groups=no FAILED
> > (ovs-macros.at:255)
>
> This test case fails for me locally too with just the first 2 patches
applied.
>
> The failure is not seen when all the patches are applied.
>
Thanks Numan. Sorry for my mistake. When fixing the system test failures I
misplaced the fix in patch 2 while they should belong to patch 5. Fixed in
v3.

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

Reply via email to