On Wed, Sep 26, 2018 at 03:53:17PM -0700, Yi-Hung Wei wrote: > On Wed, Sep 26, 2018 at 3:27 PM Ben Pfaff <[email protected]> wrote: > > > > On Wed, Sep 26, 2018 at 03:08:03PM -0700, Yi-Hung Wei wrote: > > > This patch addresses the issue that the conntrack fields associated > > > with a packet are missing after a packet is resumed by NXT_RESUME. > > > For example, the last rule in the following OpenFlow pipeline is not > > > working without this patch. > > > > > > table=0, arp,in_port=1 action=2 > > > table=0, arp,in_port=2 action=1 > > > table=0, in_port=2 icmp action=output:1 > > > table=0, in_port=1 icmp action=ct(table=1) > > > table=1, icmp action=controller(pause) resubmit(,2) > > > table=2, in_port=1 icmp ct_state=+trk+new action=output:2 > > > > > > A unit test is added to prevent regression. > > > > > > Fixes: 77ab5fd2a95b ("Implement serializing the state of packet traversal > > > in "continuations".") > > > VMware-BZ: #2202764 > > > Signed-off-by: Yi-Hung Wei <[email protected]> > > > --- > > > v1->v2: A unit test is added. > > > > Thanks for adding a test. Is there a reason why a test can't be added > > in the "normal" testsuite rather than system-tests? The regular > > testsuite runs much more often. > > Right, it makes more sense to be in the regular testsuite. > > I move the test out from system traffic test to the regular testsuite in v3. > > https://patchwork.ozlabs.org/patch/975486/
Thank you. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
