Re: [ovs-dev] [PATCH v2 9/9] system-traffic: Add zone-based conntrack timeout policy test

2019-08-20 Thread Darrell Ball
On Mon, Aug 12, 2019 at 5:22 PM Yi-Hung Wei wrote: > On Sun, Aug 11, 2019 at 12:30 PM Darrell Ball wrote: > > > > I did some further testing and ran into another issue; in this case, > one, I did not expect. > > > > I added an additional sending of packets at the end of the test after > this

Re: [ovs-dev] [PATCH v2 9/9] system-traffic: Add zone-based conntrack timeout policy test

2019-08-12 Thread Darrell Ball
On Mon, Aug 12, 2019 at 5:15 PM Yi-Hung Wei wrote: > On Sun, Aug 11, 2019 at 12:30 PM Darrell Ball wrote: > > > > I did some further testing and ran into another issue; in this case, > one, I did not expect. > > > > I added an additional sending of packets at the end of the test after > this

Re: [ovs-dev] [PATCH v2 9/9] system-traffic: Add zone-based conntrack timeout policy test

2019-08-12 Thread Darrell Ball
On Mon, Aug 12, 2019 at 5:22 PM Yi-Hung Wei wrote: > On Sun, Aug 11, 2019 at 12:30 PM Darrell Ball wrote: > > > > I did some further testing and ran into another issue; in this case, > one, I did not expect. > > > > I added an additional sending of packets at the end of the test after > this

Re: [ovs-dev] [PATCH v2 9/9] system-traffic: Add zone-based conntrack timeout policy test

2019-08-12 Thread Yi-Hung Wei
On Sun, Aug 11, 2019 at 12:30 PM Darrell Ball wrote: > > I did some further testing and ran into another issue; in this case, one, I > did not expect. > > I added an additional sending of packets at the end of the test after this > check: > > AT_CHECK([ovs-appctl dpctl/dump-conntrack |

Re: [ovs-dev] [PATCH v2 9/9] system-traffic: Add zone-based conntrack timeout policy test

2019-08-12 Thread Yi-Hung Wei
On Sun, Aug 11, 2019 at 12:30 PM Darrell Ball wrote: > > I did some further testing and ran into another issue; in this case, one, I > did not expect. > > I added an additional sending of packets at the end of the test after this > check: > > AT_CHECK([ovs-appctl dpctl/dump-conntrack |

Re: [ovs-dev] [PATCH v2 9/9] system-traffic: Add zone-based conntrack timeout policy test

2019-08-11 Thread Darrell Ball
I did some further testing and ran into another issue; in this case, one, I did not expect. I added an additional sending of packets at the end of the test after this check: AT_CHECK([ovs-appctl dpctl/dump-conntrack | FORMAT_CT(10.1.1.2)], [0], [dnl ]) Below is new code dnl Do it again dnl

Re: [ovs-dev] [PATCH v2 9/9] system-traffic: Add zone-based conntrack timeout policy test

2019-08-06 Thread Darrell Ball
On Tue, Aug 6, 2019 at 11:07 AM Yi-Hung Wei wrote: > On Tue, Aug 6, 2019 at 10:21 AM Darrell Ball wrote: > > > > > > I did some more testing and found a similar problem as in V1. > > > > This test can be run successfully once and then fails after that. > > Maybe you want to look into that. It

Re: [ovs-dev] [PATCH v2 9/9] system-traffic: Add zone-based conntrack timeout policy test

2019-08-06 Thread Yi-Hung Wei
On Tue, Aug 6, 2019 at 10:21 AM Darrell Ball wrote: > > > I did some more testing and found a similar problem as in V1. > > This test can be run successfully once and then fails after that. > Maybe you want to look into that. It is probably related to: > > dball@ubuntu:~/openvswitch/ovs$ lsmod |

Re: [ovs-dev] [PATCH v2 9/9] system-traffic: Add zone-based conntrack timeout policy test

2019-08-06 Thread Darrell Ball
On Mon, Aug 5, 2019 at 9:03 PM Darrell Ball wrote: > Thanks for the patch > > I see the test is much improved now from V1 and passes - thanks > > Ideally, tests should be associated with some code for context > It could be folded into patch 8 > I did some more testing and found a similar

Re: [ovs-dev] [PATCH v2 9/9] system-traffic: Add zone-based conntrack timeout policy test

2019-08-05 Thread Darrell Ball
Thanks for the patch I see the test is much improved now from V1 and passes - thanks Ideally, tests should be associated with some code for context It could be folded into patch 8 On Thu, Aug 1, 2019 at 3:12 PM Yi-Hung Wei wrote: > This patch adds a system traffic test to verify the

[ovs-dev] [PATCH v2 9/9] system-traffic: Add zone-based conntrack timeout policy test

2019-08-01 Thread Yi-Hung Wei
This patch adds a system traffic test to verify the zone-based conntrack timeout feature. The test uses ovs-vsctl commands to configure the customized ICMP and UDP timeout on zone 5 to a shorter period. It then injects ICMP and UDP traffic to conntrack, and checks if the corresponding conntrack