On Tue, Mar 14, 2023 at 11:29 AM Simon Horman <[email protected]> wrote:
> On Mon, Mar 13, 2023 at 03:29:19PM +0100, Ales Musil wrote: > > On Mon, Mar 13, 2023 at 1:08 PM Simon Horman <[email protected]> > > wrote: > > > On Mon, Mar 13, 2023 at 08:20:32AM +0100, Ales Musil wrote: > > > > On Fri, Mar 10, 2023 at 2:25 PM Simon Horman < > [email protected]> > > > > wrote: > > > > > On Thu, Mar 09, 2023 at 07:16:04AM +0100, Ales Musil wrote: > > ... > > > > > > I am seeing consistent failure of system-tests with this version :( > > > > > > > > > > 237: SNAT in separate zone from DNAT -- ovn-northd -- > > > parallelization=yes > > > > > -- ovn_monitor_all=yes FAILED (system-ovn.at:8702) > > > > > 238: SNAT in separate zone from DNAT -- ovn-northd -- > > > parallelization=yes > > > > > -- ovn_monitor_all=no FAILED (system-ovn.at:8702) > > > > > 239: SNAT in separate zone from DNAT -- ovn-northd -- > > > parallelization=no > > > > > -- ovn_monitor_all=yes FAILED (system-ovn.at:8702) > > > > > 240: SNAT in separate zone from DNAT -- ovn-northd -- > > > parallelization=no > > > > > -- ovn_monitor_all=no FAILED (system-ovn.at:8702) > > > > > > > > > > Link: > > > > > > > > > https://github.com/horms/ovn/actions/runs/4383139214/jobs/7676416462#step:13:3734 > > > > > > > > > > > > > > I was wondering why it was always green for me and this is the system > > > tests > > > > over userspace datapath. > > > > It's nice that we are catching bugs with that already. I will take a > look > > > > at why it is failing. But IMO it shouldn't > > > > be as this patch doesn't have anything specific per datapath type. > > > > > > It may be that there is some unreliability in the test, unrelated to > your > > > patch. If so, I wouldn't see it as blocking your patch. But it would be > > > nice to get on top of it at some point. > > > > > > > > We are actually hitting the recirculation limit in this scenario (6 for > > userspace datapath). > > I'm not yet sure how to properly solve that. > > Is it an issue with the way the test is structured, > the feature the test is exercising, > or something else? > > It looks like a combination of multiple factors: 1) We have recently started testing userspace OvS. 2) This patch redesigns the test and the way we operate with CT zones. 3) And the overall way how OVN pipeline is designed. -- Ales Musil Senior Software Engineer - OVN Core Red Hat EMEA <https://www.redhat.com> [email protected] IM: amusil <https://red.ht/sig> _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
