Re: [Ryu-devel] [PATCH 1/2] utils: Fix bytearray conversion

2014-11-18 Thread FUJITA Tomonori
On Wed, 19 Nov 2014 09:25:22 +0900 Yusuke Iwase wrote: > The parameter buf is an instance of bytearray, but Ryu tries to convert it as > string, > and outputs the error messages as a result. > This patch fixes this problem. > > Signed-off-by: IWASE Yusuke > --- > ryu/ofproto/ofproto_parser.py

Re: [Ryu-devel] help verify get flow stats for MPLS function.

2014-11-18 Thread YAMAMOTO Takashi
> On Wed, 19 Nov 2014 13:26:37 +0800 > Jalee Lee wrote: > >> I know MPLS header total 4 byte, but MPLS label just use 20bit. Could you >> tell me in the ver1.3.2 spec, which section have specify >> OFPXMT_OFB_MPLS_LABEL must be 4 byte? The Wireshark can dissector my pcap >> file. >> I need strong

Re: [Ryu-devel] help verify get flow stats for MPLS function.

2014-11-18 Thread FUJITA Tomonori
On Wed, 19 Nov 2014 13:26:37 +0800 Jalee Lee wrote: > I know MPLS header total 4 byte, but MPLS label just use 20bit. Could you > tell me in the ver1.3.2 spec, which section have specify > OFPXMT_OFB_MPLS_LABEL must be 4 byte? The Wireshark can dissector my pcap > file. > I need strong reason tel

Re: [Ryu-devel] Problem with topology view

2014-11-18 Thread FUJITA Tomonori
On Tue, 18 Nov 2014 20:18:12 -0300 "Pablo M. Guevara" wrote: > Team, > I'm currently testing Ryu with Mininet. I got a fresh copy from the > repo and I see the following issue: > > 1) When I run topology only > > $ ryu-manager --verbose --observe-links > ~/ryu/ryu/app/gui_topology/gui_topology.

Re: [Ryu-devel] [PATCH 0/2] test_ofctl_v1_2/3 improvement

2014-11-18 Thread FUJITA Tomonori
On Tue, 18 Nov 2014 13:25:07 +0900 Minoru TAKAHASHI wrote: > following patches improve test_ofctl_v1_2/3. > > Minoru TAKAHASHI (2): > test_ofctl_v1_2/3: Fix some test cases > test_ofctl_v1_2/3: Add a test case of masked metadata > > ryu/tests/unit/lib/test_ofctl.py | 39 +++

Re: [Ryu-devel] help verify get flow stats for MPLS function.

2014-11-18 Thread Jalee Lee
Hi Yusuke, Thanks your answer. I know MPLS header total 4 byte, but MPLS label just use 20bit. Could you tell me in the ver1.3.2 spec, which section have specify OFPXMT_OFB_MPLS_LABEL must be 4 byte? The Wireshark can dissector my pcap file. I need strong reason tell my vender change them code. If

Re: [Ryu-devel] help verify get flow stats for MPLS function.

2014-11-18 Thread Yusuke Iwase
Hi, I checked your attached file, I think the switch sent a malformed message. Please check the attached file which is the test result of Open vSwitch. oxm_length of OFPXMT_OFB_MPLS_LABEL in OFPAT_SET_FIELD is wrong. oxm_length in your file is 3. I guess oxm_length of OFPXMT_OFB_MPLS_LABEL must

Re: [Ryu-devel] ofctl_rest.py add flow to flood packets OF v1.3

2014-11-18 Thread Minoru TAKAHASHI
Hi, Pablo On 2014年11月19日 10:07, Pablo M. Guevara wrote: > Hello Team, > I'm trying to add a flow to flood packets when coming in on a given port. I > tried all these options but I always get the same result, packet dropped. > > curl -X POST -d '{"dpid": "1","match": {"in_port":"1"}, "hard_timeou

[Ryu-devel] ofctl_rest.py add flow to flood packets OF v1.3

2014-11-18 Thread Pablo M. Guevara
Hello Team, I'm trying to add a flow to flood packets when coming in on a given port. I tried all these options but I always get the same result, packet dropped. curl -X POST -d '{"dpid": "1","match": {"in_port":"1"}, "hard_timeout": "0", "idle_timeout": "0", "actions": [{"type": "OFPP_FLOOD"

[Ryu-devel] [PATCH 2/2] test_utils: Add unit test for ryu.utils

2014-11-18 Thread Yusuke Iwase
Signed-off-by: IWASE Yusuke --- ryu/tests/unit/test_utils.py | 49 1 file changed, 49 insertions(+) create mode 100644 ryu/tests/unit/test_utils.py diff --git a/ryu/tests/unit/test_utils.py b/ryu/tests/unit/test_utils.py new file mode 100644 index 00

[Ryu-devel] [PATCH 1/2] utils: Fix bytearray conversion

2014-11-18 Thread Yusuke Iwase
The parameter buf is an instance of bytearray, but Ryu tries to convert it as string, and outputs the error messages as a result. This patch fixes this problem. Signed-off-by: IWASE Yusuke --- ryu/ofproto/ofproto_parser.py | 2 +- ryu/utils.py | 17 ++--- 2 files c

[Ryu-devel] Problem with topology view

2014-11-18 Thread Pablo M. Guevara
Team, I'm currently testing Ryu with Mininet. I got a fresh copy from the repo and I see the following issue: 1) When I run topology only $ ryu-manager --verbose --observe-links ~/ryu/ryu/app/gui_topology/gui_topology.py I can see the topology ok. 2) When I run with simple switch, I only