Re: [ovs-dev] [PATCH 1/2] tests/system-offloads-traffic.at: add sanity check

2017-08-18 Thread Joe Stringer
On 17 August 2017 at 11:07, Joe Stringer wrote: > On 17 August 2017 at 00:17, Roi Dayan wrote: >> >> >> On 17/08/2017 08:32, Roi Dayan wrote: >>> >>> >>> >>> On 17/08/2017 01:17, Joe Stringer wrote: On 16 August 2017 at 05:14, Roi Dayan wrote: > > Doing dump-flows also altering

Re: [ovs-dev] [PATCH 1/2] tests/system-offloads-traffic.at: add sanity check

2017-08-17 Thread Joe Stringer
On 17 August 2017 at 00:17, Roi Dayan wrote: > > > On 17/08/2017 08:32, Roi Dayan wrote: >> >> >> >> On 17/08/2017 01:17, Joe Stringer wrote: >>> >>> On 16 August 2017 at 05:14, Roi Dayan wrote: Doing dump-flows also altering the netdev ports list. So doing it pre the actual test i

Re: [ovs-dev] [PATCH 1/2] tests/system-offloads-traffic.at: add sanity check

2017-08-17 Thread Roi Dayan
On 17/08/2017 08:32, Roi Dayan wrote: On 17/08/2017 01:17, Joe Stringer wrote: On 16 August 2017 at 05:14, Roi Dayan wrote: Doing dump-flows also altering the netdev ports list. So doing it pre the actual test is adding a check to make sure we don't break the that list. Signed-off-by: Roi

Re: [ovs-dev] [PATCH 1/2] tests/system-offloads-traffic.at: add sanity check

2017-08-16 Thread Roi Dayan
On 17/08/2017 01:17, Joe Stringer wrote: On 16 August 2017 at 05:14, Roi Dayan wrote: Doing dump-flows also altering the netdev ports list. So doing it pre the actual test is adding a check to make sure we don't break the that list. Signed-off-by: Roi Dayan Reviewed-by: Paul Blakey I'm a

Re: [ovs-dev] [PATCH 1/2] tests/system-offloads-traffic.at: add sanity check

2017-08-16 Thread Joe Stringer
On 16 August 2017 at 05:14, Roi Dayan wrote: > Doing dump-flows also altering the netdev ports list. > So doing it pre the actual test is adding a check to > make sure we don't break the that list. > > Signed-off-by: Roi Dayan > Reviewed-by: Paul Blakey I'm actually not sure what the requiremen

[ovs-dev] [PATCH 1/2] tests/system-offloads-traffic.at: add sanity check

2017-08-16 Thread Roi Dayan
Doing dump-flows also altering the netdev ports list. So doing it pre the actual test is adding a check to make sure we don't break the that list. Signed-off-by: Roi Dayan Reviewed-by: Paul Blakey --- tests/system-offloads-traffic.at | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/sys