Re: [ovs-dev] [PATCH v2 14/15] system-tests: Add tcp simple test.

2016-04-27 Thread Daniele Di Proietto 


On 26/04/2016 16:38, "Flavio Leitner"  wrote:

>On Fri, Apr 15, 2016 at 05:02:46PM -0700, Daniele Di Proietto wrote:
>> Useful to test the datapath ability to forward tcp packets without the
>> complexity of connection tracking.
>> 
>> Signed-off-by: Daniele Di Proietto 
>> ---
>
>This patch can be separated from this patchset as well.

Agreed, I'll push that separately

>
>Thanks for improving the testsuite.
>
>Acked-by: Flavio Leitner 

Thanks for the reviews!
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v2 14/15] system-tests: Add tcp simple test.

2016-04-26 Thread Flavio Leitner 
On Fri, Apr 15, 2016 at 05:02:46PM -0700, Daniele Di Proietto wrote:
> Useful to test the datapath ability to forward tcp packets without the
> complexity of connection tracking.
> 
> Signed-off-by: Daniele Di Proietto 
> ---

This patch can be separated from this patchset as well.

Thanks for improving the testsuite.

Acked-by: Flavio Leitner 


___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v2 14/15] system-tests: Add tcp simple test.

2016-04-19 Thread Joe Stringer 
On 15 April 2016 at 17:02, Daniele Di Proietto  wrote:
> Useful to test the datapath ability to forward tcp packets without the
> complexity of connection tracking.
>
> Signed-off-by: Daniele Di Proietto 

I think we could do with a few more of these incremental steps on the
way up to the more complex configurations in the existing tests. This
is a step in the right direction.

Acked-by: Joe Stringer 
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH v2 14/15] system-tests: Add tcp simple test.

2016-04-15 Thread Daniele Di Proietto 
Useful to test the datapath ability to forward tcp packets without the
complexity of connection tracking.

Signed-off-by: Daniele Di Proietto 
---
 tests/system-traffic.at | 20 
 1 file changed, 20 insertions(+)

diff --git a/tests/system-traffic.at b/tests/system-traffic.at
index dceae15..c8fbe0d 100644
--- a/tests/system-traffic.at
+++ b/tests/system-traffic.at
@@ -23,6 +23,26 @@ NS_CHECK_EXEC([at_ns0], [ping -s 3200 -q -c 3 -i 0.3 -w 2 
10.1.1.2 | FORMAT_PING
 OVS_TRAFFIC_VSWITCHD_STOP
 AT_CLEANUP
 
+AT_SETUP([datapath - http between two ports])
+OVS_TRAFFIC_VSWITCHD_START()
+
+AT_CHECK([ovs-ofctl add-flow br0 "actions=normal"])
+
+ADD_NAMESPACES(at_ns0, at_ns1)
+
+ADD_VETH(p0, at_ns0, br0, "10.1.1.1/24")
+ADD_VETH(p1, at_ns1, br0, "10.1.1.2/24")
+
+NS_CHECK_EXEC([at_ns0], [ping -q -c 3 -i 0.3 -w 2 10.1.1.2 | FORMAT_PING], 
[0], [dnl
+3 packets transmitted, 3 received, 0% packet loss, time 0ms
+])
+
+NETNS_DAEMONIZE([at_ns1], [[$PYTHON $srcdir/test-l7.py]], [http0.pid])
+NS_CHECK_EXEC([at_ns0], [wget 10.1.1.2 -t 3 -T 1 --retry-connrefused -v -o 
wget0.log])
+
+OVS_TRAFFIC_VSWITCHD_STOP
+AT_CLEANUP
+
 AT_SETUP([datapath - ping between two ports on vlan])
 OVS_TRAFFIC_VSWITCHD_START()
 
-- 
2.1.4

___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev