[Ryu-devel] [PATCH 14/14] sw test tool: Add default test files (of14: new features)

2014-06-22 Thread Yuichi Ito
atch' of FlowMod message. Signed-off-by: Yuichi Ito --- .../of14/action/25_SET_FIELD/41_PBB_UCA.json | 235 ryu/tests/switch/of14/match/41_PBB_UCA.json| 589 2 files changed, 824 insertions(+) create mode 100644 ryu/tests/switch/of14/action/25_SET_FIEL

[Ryu-devel] [PATCH 07/14] sw test tool: Modify OFPActionSetField to normalize

2014-06-22 Thread Yuichi Ito
::::' are not normalized to 'ff::'. Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 39 +++ 1 file changed, 39 insertions(+) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index 73a7bfb..b6826e5

[Ryu-devel] [PATCH 08/14] sw test tool: run_mininet: support Open vSwitch with OpenFlow 1.4

2014-06-22 Thread Yuichi Ito
/switch/run_mininet.py Signed-off-by: Yuichi Ito --- ryu/tests/switch/run_mininet.py | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/ryu/tests/switch/run_mininet.py b/ryu/tests/switch/run_mininet.py index 9dce788..333efcb 100755 --- a/ryu/tests/switch/run_mininet.py

[Ryu-devel] [PATCH 06/14] sw test tool: Modify OFPMatch to normalize

2014-06-22 Thread Yuichi Ito
0:' are not normalized to 'ff::'. Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 16 1 file changed, 16 insertions(+) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index 9cf65e9..73a7bfb 100644 --- a/ryu/tests/switch/tester.py ++

[Ryu-devel] [PATCH 05/14] sw test tool: Enable using connected switches to compare

2014-06-22 Thread Yuichi Ito
Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 43 ++- 1 file changed, 30 insertions(+), 13 deletions(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index d3e22d3..9cf65e9 100644 --- a/ryu/tests/switch/tester.py

[Ryu-devel] [PATCH 04/14] sw test tool: Enable using user-specified versions to parse

2014-06-22 Thread Yuichi Ito
Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 22 +- 1 file changed, 17 insertions(+), 5 deletions(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index d104a58..d3e22d3 100644 --- a/ryu/tests/switch/tester.py +++ b/ryu/tests/switch

[Ryu-devel] [PATCH 03/14] sw test tool: Add support for using user-specified versions

2014-06-22 Thread Yuichi Ito
Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 56 +--- 1 file changed, 48 insertions(+), 8 deletions(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index 98788a9..d104a58 100644 --- a/ryu/tests/switch/tester.py

[Ryu-devel] [PATCH 02/14] ryu/flags: Add config parameters related sw test tool

2014-06-22 Thread Yuichi Ito
Signed-off-by: Yuichi Ito --- ryu/flags.py |8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/ryu/flags.py b/ryu/flags.py index 98cb96c..7c50e69 100644 --- a/ryu/flags.py +++ b/ryu/flags.py @@ -56,5 +56,11 @@ CONF.register_cli_opts([ cfg.StrOpt('target'

[Ryu-devel] [PATCH 00/14] sw test tool: Add support for OpenFlow 1.4

2014-06-22 Thread Yuichi Ito
#x27;openflow14' (case-insensitive). Both default values are 'openflow13'. ex) ryu-manager ryu/tests/switch/tester.py --test-switch-dir ryu/tests/switch/of14/ --test-switch-target-version openflow14 Yuichi Ito (14): sw test tool: Modify conditions of ofp_packet_in_reason ry

[Ryu-devel] [PATCH 01/14] sw test tool: Modify conditions of ofp_packet_in_reason

2014-06-22 Thread Yuichi Ito
to controller in packet-out. */ }; Therefore, "reason != OFPR_ACTION" means "reason == OFPR_NOMATCH or reason == OFPR_INVALID_TTL". NOTE: OFPR_TABLE_MISS is defined as OFPR_NO_MATCH in ryu.ofproto.ofproto_v1_4. Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py

[Ryu-devel] [PATCH] sw test tool: Enable IPv6 flow label maskable

2014-06-17 Thread Yuichi Ito
OF 1.3.4 spec (B.15.1 Changes) says: Make IPv6 flow label maskable (EXT-101). This patch makes IPv6 flow label maskable, and adds a test file for IPv6 flow label with masks. Signed-off-by: Yuichi Ito --- .../switch/of13/match/28_IPV6_FLABEL_Mask.json | 927 ryu

[Ryu-devel] [PATCH 3/4] sw test tool: Improve readability of MAC mask values

2014-06-08 Thread Yuichi Ito
1099511627775 -> "00:ff:ff:ff:ff:ff" 281474976710400 -> "ff:ff:ff:ff:ff:00" 281474959998975 -> "ff:ff:ff:00:ff:ff" 281470698520575 -> "ff:ff:00:ff:ff:ff" Signed-off-by: Yuichi Ito --- ryu/tests/switch/of13/match/03_

[Ryu-devel] [PATCH 0/4] sw test tool: Improve readability of test files

2014-06-08 Thread Yuichi Ito
Yuichi Ito (4): sw test tool: Improve readability of tcp options sw test tool: Improve readability of padding data sw test tool: Improve readability of MAC mask values sw test tool: Improve readability of IP mask values ryu/tests/switch/of13/action/00_OUTPUT.json| 12 +-- ryu

Re: [Ryu-devel] ryu-IGMP

2014-06-08 Thread Yuichi Ito
message. Everyone receiving message > but multicasting is not working. > Please find the topology and igmp log file in the attachment. > > > > > > On Tue, Jun 3, 2014 at 10:11 PM, Yuichi Ito <mailto:ito.yuic...@gmail.com>> wrote: > > Hi Amjad, > > >

Re: [Ryu-devel] ryu-IGMP

2014-06-03 Thread Yuichi Ito
sing ryu Igmp. > > Please find the attachments: > 1. Command outputs > 2. Mininet topology I am using, ( running controller as remotely with > loopback IP)/ so, consider the topology without VM in the previous topology > in PNG file. > 3. Output log file : igmp.log &g

Re: [Ryu-devel] ryu-IGMP

2014-06-03 Thread Yuichi Ito
ts of 'ifconfig' of each hosts - network topology (same as former topology.png?) And could you give me a whole log file? To output a log file, please append options as follows: # ryu-manager ryu.app.simple_switch_igmp --verbose --log-file=igmp.log On Tue, 03 Jun 2014 15:15:51 +0900 Yuichi It

Re: [Ryu-devel] ryu-IGMP

2014-06-02 Thread Yuichi Ito
NT igmplib->SimpleSwitchIgmp EventPacketIn > packet in 138641775921991 02:8a:bb:7f:54:5c 33:33:00:00:00:01 3 > EVENT ofp_event->igmplib EventOFPPacketIn > EVENT igmplib->SimpleSwitchIgmp EventPacketIn > packet in 196119712417347 02:8a:bb:7f:54:5c 33:33:00:00:00:01 2 > EVENT ofp_ev

Re: [Ryu-devel] ryu-IGMP

2014-06-02 Thread Yuichi Ito
On Mon, 2 Jun 2014 23:40:12 -0400 Md. Amjad Hossain wrote: > Hello Yuichi Ito, > > Sorry for replying after long time. Actually I had to stop working for my > physical condition and other reasons. > > Thank you for your suggestions. I want to start my work again. Could you &

[Ryu-devel] [PATCH/RFC] of14: Fix an issue about converting JSON with unicode

2014-06-02 Thread Yuichi Ito
7;{"name":"test"}') >>> print OFPTableFeaturesStats.from_jsondict(json_feature) OFPTableFeaturesStats(config=None,length=None,max_entries=None,metadata_match=None,metadata_write=None,name='test',properties=None,table_id=None) I think that from_js

[Ryu-devel] [PATCH 6/7] sw test tool: Enable sending packets using random values

2014-05-26 Thread Yuichi Ito
"OXMTlv":{ "field":"in_port", "value":2 } } ] }, "kbps":1500 } ]

[Ryu-devel] [PATCH 5/7] sw test tool: Enable parsing OFPGroupMod messages in 'prerequisite'

2014-05-26 Thread Yuichi Ito
Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index 5765212..eacd3fb 100644 --- a/ryu/tests/switch/tester.py +++ b/ryu/tests/switch/tester.py @@ -99,6 +99,7

[Ryu-devel] [PATCH 4/7] sw test tool: Add support for confirmation of installed group entries

2014-05-26 Thread Yuichi Ito
Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 41 ++--- 1 file changed, 38 insertions(+), 3 deletions(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index 674f8f1..5765212 100644 --- a/ryu/tests/switch/tester.py +++ b

[Ryu-devel] [PATCH 3/7] sw test tool: Add support for installation of group tables

2014-05-26 Thread Yuichi Ito
Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py |8 1 file changed, 8 insertions(+) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index 1f8264d..674f8f1 100644 --- a/ryu/tests/switch/tester.py +++ b/ryu/tests/switch/tester.py @@ -132,6 +132,7

[Ryu-devel] [PATCH 2/7] sw test tool: Add support for initialization of group tables

2014-05-26 Thread Yuichi Ito
Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 13 + 1 file changed, 13 insertions(+) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index ebc3e3d..1f8264d 100644 --- a/ryu/tests/switch/tester.py +++ b/ryu/tests/switch/tester.py @@ -131,6 +131,7

[Ryu-devel] [PATCH 1/7] sw test tool: Add a link between switches

2014-05-26 Thread Yuichi Ito
This change is for tests that use group_table. Signed-off-by: Yuichi Ito --- ryu/tests/switch/run_mininet.py |1 + ryu/tests/switch/tester.py | 18 +- 2 files changed, 10 insertions(+), 9 deletions(-) diff --git a/ryu/tests/switch/run_mininet.py b/ryu/tests/switch

[Ryu-devel] [PATCH 0/7] sw test tool: Enable using group tables

2014-05-26 Thread Yuichi Ito
"OFPInstructionActions":{ "actions":[ { "OFPActionGroup":{ "group_id":0 }

Re: [Ryu-devel] REST_Router API

2014-05-21 Thread Yuichi Ito
Hi, Please enclose JSON string with single quotations(ascii code is 0x27) like: curl -X POST -d '{"address":"172.16.20.1/24"}' http://localhost:8080/router/0001 I think you are using "right single quotation marks", 0x2019 at unicode. On Thu, 22 May 2014 06:14:19 +0500 Zubair Hafeez

[Ryu-devel] [PATCH 7/7] sw test tool: pylint

2014-05-21 Thread Yuichi Ito
methods are never called. This patch is only for avoiding pylint error. Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py |6 ++ 1 file changed, 6 insertions(+) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index 8622cba..aef92b6 100644 --- a/ryu/tests

[Ryu-devel] [PATCH 6/7] sw test tool: Reduce similar methods of checking entries exist

2014-05-21 Thread Yuichi Ito
All of _test_xxx_exist_chk() are merged as _test_exist_chk(). Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 57 +++- 1 file changed, 30 insertions(+), 27 deletions(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py

[Ryu-devel] [PATCH 5/7] sw test tool: Reduce similar methods of handling events

2014-05-21 Thread Yuichi Ito
All of xxx_stats_reply_handler() are merged as stats_reply_handler(). Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 61 1 file changed, 22 insertions(+), 39 deletions(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch

[Ryu-devel] [PATCH 4/7] sw test tool: Reduce similar methods of initializing flows

2014-05-21 Thread Yuichi Ito
_test_initialize_flow_tester() is merged into _test_initialize_flow(). Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 25 +++-- 1 file changed, 7 insertions(+), 18 deletions(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index e715832

[Ryu-devel] [PATCH 3/7] sw test tool: Reduce similar methods of deleting flows

2014-05-21 Thread Yuichi Ito
OpenFlowSw.del_test_flow() and OpenFlowSw.del_flows_for_throughput_analysis() are merged as OpenFlowSw.del_flows(). And rename OpenFlowSw.del_test_meter() to OpenFlowSw.del_meters(), like OpenFlowSw.del_flows(). Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 28

[Ryu-devel] [PATCH 2/7] sw test tool: Reduce similar classes

2014-05-21 Thread Yuichi Ito
TargetSw and TesterSw are merged as OpenFlowSw. Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 82 +++- 1 file changed, 36 insertions(+), 46 deletions(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index 7468c74

[Ryu-devel] [PATCH 1/7] sw test tool: Remove an unnecessary wrapper method

2014-05-21 Thread Yuichi Ito
Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py |5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index c3830fc..7468c74 100644 --- a/ryu/tests/switch/tester.py +++ b/ryu/tests/switch/tester.py @@ -492,7

[Ryu-devel] [PATCH 0/7] sw test tool: Refine codes

2014-05-21 Thread Yuichi Ito
Yuichi Ito (7): sw test tool: Remove an unnecessary wrapper method sw test tool: Reduce similar classes sw test tool: Reduce similar methods of deleting flows sw test tool: Reduce similar methods of initializing flows sw test tool: Reduce similar methods of handling events sw test tool

Re: [Ryu-devel] [PATCH 1/2] tester: ok_count should be increased sequentially with each successful test.

2014-05-14 Thread Yuichi Ito
On Wed, 14 May 2014 21:48:18 +0900 Chiu,Cheng-Han wrote: > From: "Chiu,Cheng-Han" > > Signed-off-by: Chiu,Cheng-Han > --- > ryu/tests/switch/tester.py |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py > index a

Re: [Ryu-devel] [PATCH 2/2] tester: let each attribute of meter had a chance to be compared.

2014-05-14 Thread Yuichi Ito
On Wed, 14 May 2014 21:48:19 +0900 Chiu,Cheng-Han wrote: > From: "Chiu,Cheng-Han" > > Signed-off-by: Chiu,Cheng-Han > --- > ryu/tests/switch/tester.py |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py > index 4

[Ryu-devel] [PATCH] set default attributes for OFPPortMod

2014-05-07 Thread Yuichi Ito
Signed-off-by: Yuichi Ito --- ryu/ofproto/ofproto_v1_0_parser.py |3 ++- ryu/ofproto/ofproto_v1_2_parser.py |3 ++- ryu/ofproto/ofproto_v1_3_parser.py |3 ++- ryu/ofproto/ofproto_v1_4_parser.py |5 +++-- 4 files changed, 9 insertions(+), 5 deletions(-) diff --git a/ryu/ofproto

[Ryu-devel] [PATCH 3/3] sw test tool: pylint

2014-05-07 Thread Yuichi Ito
W:546,14: Access to a protected member _send_msg of a client class (protected-access) Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch

[Ryu-devel] [PATCH 2/3] sw test tool: Remove unused codes

2014-05-07 Thread Yuichi Ito
Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 23 ++- 1 file changed, 10 insertions(+), 13 deletions(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index 68d0ec8..135b036 100644 --- a/ryu/tests/switch/tester.py +++ b/ryu/tests/switch

[Ryu-devel] [PATCH 1/3] sw test tool: Reduce similar methods

2014-05-07 Thread Yuichi Ito
- _test_flow_install() - _test_meter_install() are merged as - _test_msg_install() Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 24 ++-- 1 file changed, 6 insertions(+), 18 deletions(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch

[Ryu-devel] [PATCH 0/3] sw test tool: refine codes

2014-05-07 Thread Yuichi Ito
Yuichi Ito (3): sw test tool: Reduce similar methods sw test tool: Remove unused codes sw test tool: pylint ryu/tests/switch/tester.py | 69 +--- 1 file changed, 27 insertions(+), 42 deletions(-) -- 1.7.10.4

[Ryu-devel] [PATCH 0/2] sw test tool: Fix test files

2014-04-25 Thread Yuichi Ito
set provides a reverting patch and a correct patch. - Revert changes related to MAC frames which use VLAN, MPLS, and PBB - Resize pad fields related to POP_MPLS/POP_PBB Yuichi Ito (2): sw test tool: Revert changes of test files sw test tool: Modify lengths of pad fields for ARP ryu/tests

[Ryu-devel] [PATCH] sw test tool: Fix test files

2014-04-24 Thread Yuichi Ito
This patch is after Fixing-MAC-addresses patch. Yuichi Ito (1): sw test tool: Modify lengths of pad fields for ARP ryu/tests/switch/of13/action/00_OUTPUT.json|4 +- ryu/tests/switch/of13/action/15_SET_MPLS_TTL.json |4 +- ryu/tests/switch/of13/action/16_DEC_MPLS_TTL.json

Re: [Ryu-devel] SW Tests: Incorrect packets sent to target switch

2014-04-23 Thread Yuichi Ito
Hi, thank you for reporting again. I'm creating a fix patch now. Please wait for a while. On Tue, 22 Apr 2014 17:43:01 +0800 arne_goetje wrote: > Dear all, > > I found that some tests send incorrect packets on the data plane to the > target switch. This can cause problems, as incorrect packets

[Ryu-devel] [PATCH] sw test tool: Fix to compare OFPMatch using masks without byte boundary

2014-04-18 Thread Yuichi Ito
Mask lengths without byte boundary are fixed as follows: - VLAN_VID, from 16 bits (2 bytes) to 12 + 1 bits - IPV6_EXTHDR, from 16 bits (2 bytes) to 9 bits Reported-by: Arne Goetje Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 15 --- 1 file changed, 12 insertions

Re: [Ryu-devel] [PATCH 1/3] sw test tool: Fix to compare OFPMatch ignoring masks that are all one bits

2014-04-17 Thread Yuichi Ito
est Regards! > Arne Goetje (高盛華) > > > > From: FUJITA Tomonori > To: ito.yuic...@gmail.com, > Cc: ryu-devel@lists.sourceforge.net > Date: 04/15/2014 03:09 PM > Subject:Re: [Ryu-devel] [PATCH 1/3] sw test tool: Fix to compare > OFPMatch ignoring masks that

[Ryu-devel] [PATCH 1/3] of12: set default attributes for group-related classes

2014-04-15 Thread Yuichi Ito
Signed-off-by: Yuichi Ito --- ryu/ofproto/ofproto_v1_2_parser.py |8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/ryu/ofproto/ofproto_v1_2_parser.py b/ryu/ofproto/ofproto_v1_2_parser.py index 08b3ae8..2b2d48f 100644 --- a/ryu/ofproto/ofproto_v1_2_parser.py +++ b

[Ryu-devel] [PATCH 3/3] of14: set default attributes for group-related classes

2014-04-15 Thread Yuichi Ito
Signed-off-by: Yuichi Ito --- ryu/ofproto/ofproto_v1_4_parser.py |8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/ryu/ofproto/ofproto_v1_4_parser.py b/ryu/ofproto/ofproto_v1_4_parser.py index 230ac9e..618b676 100644 --- a/ryu/ofproto/ofproto_v1_4_parser.py +++ b

[Ryu-devel] [PATCH 2/3] of13: set default attributes for group-related classes

2014-04-15 Thread Yuichi Ito
Signed-off-by: Yuichi Ito --- ryu/ofproto/ofproto_v1_3_parser.py |8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/ryu/ofproto/ofproto_v1_3_parser.py b/ryu/ofproto/ofproto_v1_3_parser.py index 38eeb1f..13bf385 100644 --- a/ryu/ofproto/ofproto_v1_3_parser.py +++ b

[Ryu-devel] [PATCH 2/3] sw test tool: Fix to compare OFPMatch ignoring fields that masks are all zero bits

2014-04-14 Thread Yuichi Ito
are all zero bits. Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py |6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index dd05050..4b12e29 100644 --- a/ryu/tests/switch/tester.py +++ b/ryu/tests/switch

[Ryu-devel] [PATCH 3/3] sw test tool: Correct the comparison method to treat matches as the set

2014-04-14 Thread Yuichi Ito
). According to OF spec, the match fields are treated as the set. This means that the order of matches which flow_stats message returns is inconsistant. This patch corrects the comparison method of matches by sorting in a particular order before comparison. Signed-off-by: Yuichi Ito --- ryu/tests

[Ryu-devel] [PATCH 1/3] sw test tool: Fix to compare OFPMatch ignoring masks that are all one bits

2014-04-14 Thread Yuichi Ito
one bits. Reported-by: Arne Goetje Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 21 + 1 file changed, 21 insertions(+) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index 631412a..dd05050 100644 --- a/ryu/tests/switch/tester.py +++ b

[Ryu-devel] [PATCH] igmplib: Fix a problem that querier stops sending a query message

2014-04-06 Thread Yuichi Ito
When QUERY timeout expires, a variable for flooding is overwritten unexpectedly. So querier stops sending a query message any longer. This patch fixes the problem. Signed-off-by: Yuichi Ito --- ryu/lib/igmplib.py |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ryu/lib

Re: [Ryu-devel] Add a flow with two actions REST API

2014-04-01 Thread Yuichi Ito
Hi, please try this. {"cookie":"1","dpid":"1", "match { "dl_type":"2048", "nw_proto":"1", "priority":"5", "nw_src":"10.0.0.1", "nw_dst":"10.0.0.5", "in_port":"1"}, "actions":[{"type":"OUTPUT","port":"2"}, {"type":"SET

[Ryu-devel] [PATCH] ofctl_v1_3: Fix to enable using meter flags multiply

2014-03-30 Thread Yuichi Ito
uot;DROP", "rate": 1000}]}' http://localhost:8080/stats/meterentry/add And now enabled to show flags as follows: curl http://localhost:8080/stats/meterconfig/1 { "1": [ {"bands": [{"burst_size": 0, "rate": 1000, "type&qu

Re: [Ryu-devel] how to specify multiple bits in the flags of METER_MOD message?

2014-03-30 Thread Yuichi Ito
Hi, thank you for reporting this. I will post a patch that fixes this issue. On Sat, 29 Mar 2014 07:40:45 -0700 cheers wrote: > Thank you for the response. It may work but what i need is how i can specify > multiple flags in HTTP URL via curl. > > > -Peter > > > On Mar 29, 2014, at 2:20 AM

[Ryu-devel] [PATCH] stplib: reduce unnecessary constant

2014-03-27 Thread Yuichi Ito
and correct the inaccurate dispatchers. Signed-off-by: Yuichi Ito --- ryu/app/simple_switch_stp.py |7 --- ryu/lib/stplib.py|3 --- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/ryu/app/simple_switch_stp.py b/ryu/app/simple_switch_stp.py index 8efad14

[Ryu-devel] [PATCH 2/6] sw test tool: Add support for deletion of flows for throughput analysis

2014-03-27 Thread Yuichi Ito
Signed-off-by: WATANABE Fumitaka Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 34 ++ 1 file changed, 34 insertions(+) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index 288b23d..b6f9141 100644 --- a/ryu/tests/switch

[Ryu-devel] [PATCH 6/6] sw test tool: Add support for throughput analysis

2014-03-27 Thread Yuichi Ito
Signed-off-by: WATANABE Fumitaka Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 49 +++- 1 file changed, 48 insertions(+), 1 deletion(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index 0e172a4..631412a 100644

[Ryu-devel] [PATCH 4/6] sw test tool: Add support for getting throughput

2014-03-27 Thread Yuichi Ito
Signed-off-by: WATANABE Fumitaka Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 26 -- 1 file changed, 24 insertions(+), 2 deletions(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index 9cf4591..8b33058 100644 --- a/ryu/tests

[Ryu-devel] [PATCH 5/6] sw test tool: Enable throughput analysis

2014-03-27 Thread Yuichi Ito
Signed-off-by: WATANABE Fumitaka Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py |5 + 1 file changed, 5 insertions(+) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index 8b33058..0e172a4 100644 --- a/ryu/tests/switch/tester.py +++ b/ryu/tests/switch

[Ryu-devel] [PATCH 3/6] sw test tool: Add support for installation of flows for throughput analysis

2014-03-27 Thread Yuichi Ito
Signed-off-by: WATANABE Fumitaka Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 56 ++-- 1 file changed, 39 insertions(+), 17 deletions(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index b6f9141..9cf4591 100644

[Ryu-devel] [PATCH 1/6] sw test tool: Enable parsing a new 'egress' format

2014-03-27 Thread Yuichi Ito
The format is for throughput analysis. Signed-off-by: WATANABE Fumitaka Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 30 +- 1 file changed, 29 insertions(+), 1 deletion(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index

[Ryu-devel] [PATCH 0/6] sw test tool: Enable throughput analysis

2014-03-27 Thread Yuichi Ito
ckets that match to the condition during receiving packets to calculate throughput. Expected throughput is one of the following: - 'kbps': the throughput in kilobits per second. - 'pktps': the throughput in packets per second. The tool reports errors when the throughput is

[Ryu-devel] [PATCH 5/5] sw test tool: Enable sending packets continuously

2014-03-27 Thread Yuichi Ito
Signed-off-by: WATANABE Fumitaka Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 23 ++- 1 file changed, 18 insertions(+), 5 deletions(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index b95417a..c804927 100644 --- a/ryu/tests/switch

[Ryu-devel] [PATCH 4/5] sw test tool: Rename STATE_UNMATCH_PKT_SEND to STATE_SEND_BARRIER

2014-03-27 Thread Yuichi Ito
And rename _test_unmatch_packet_send() to _test_send_barrier(). Signed-off-by: WATANABE Fumitaka Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py

[Ryu-devel] [PATCH 3/5] sw test tool: Refactor packet sending processes

2014-03-27 Thread Yuichi Ito
Methods of STATE_FLOW_MATCH_CHK and STATE_UNMATCH_PACKET_SEND have similar processes that send a packet. This patch refactors this redundancy. Signed-off-by: WATANABE Fumitaka Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 19 +++ 1 file changed, 7 insertions

[Ryu-devel] [PATCH 1/5] sw test tool: Enable parsing a new 'ingress' format

2014-03-27 Thread Yuichi Ito
The format is for sending packets continuously. Signed-off-by: WATANABE Fumitaka Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 21 - 1 file changed, 20 insertions(+), 1 deletion(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index

[Ryu-devel] [PATCH 0/5] sw test tool: Enable sending packets continuously

2014-03-27 Thread Yuichi Ito
ype=2048)", "ipv4(proto=6)", "tcp()", "str(\"\\x11\" * (1500 - 54))" ] } ] 'data' is a packet that is sent continuously. 'pktps' is the number of packets that is sent per second (def

[Ryu-devel] [PATCH 2/5] sw test tool: Add support for sending packets continuously

2014-03-27 Thread Yuichi Ito
Signed-off-by: WATANABE Fumitaka Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 75 +++- 1 file changed, 74 insertions(+), 1 deletion(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index b43cb9f..29c8375 100644

[Ryu-devel] [PATCH 3/5] sw test tool: Add support for installation of meter entries

2014-03-27 Thread Yuichi Ito
Signed-off-by: WATANABE Fumitaka Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 25 +++-- 1 file changed, 23 insertions(+), 2 deletions(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index 984b0b4..1123e16 100644 --- a/ryu/tests/switch

[Ryu-devel] [PATCH 2/5] sw test tool: Add support for initialization of meter tables

2014-03-27 Thread Yuichi Ito
Signed-off-by: WATANABE Fumitaka Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 17 + 1 file changed, 17 insertions(+) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index 1a119e0..984b0b4 100644 --- a/ryu/tests/switch/tester.py +++ b/ryu

[Ryu-devel] [PATCH 0/5] sw test tool: Enable using meter tables

2014-03-27 Thread Yuichi Ito
op":{ "rate":1000 } } ] } } ] Yuichi Ito (5): sw test tool: Rename STATE_INIT to STATE_INIT_FLOW sw test tool: Add support for initialization of meter tables sw test t

[Ryu-devel] [PATCH 5/5] sw test tool: Enable parsing OFPMeterMod messages in 'prerequisite'

2014-03-27 Thread Yuichi Ito
Signed-off-by: WATANABE Fumitaka Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index 4d85741..171d552 100644 --- a/ryu/tests/switch/tester.py

[Ryu-devel] [PATCH 4/5] sw test tool: Add support for confirmation of installed meter entries

2014-03-27 Thread Yuichi Ito
Signed-off-by: WATANABE Fumitaka Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 55 1 file changed, 55 insertions(+) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index 1123e16..4d85741 100644 --- a/ryu/tests

[Ryu-devel] [PATCH 1/5] sw test tool: Rename STATE_INIT to STATE_INIT_FLOW

2014-03-27 Thread Yuichi Ito
And rename _test_initialize() to _test_initialize_flow(). Signed-off-by: WATANABE Fumitaka Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py index

[Ryu-devel] [PATCH] sw test tool: Fix a problem for disconnection

2014-03-09 Thread Yuichi Ito
Disconnection of switches has caused a RyuInternalError because target_sw/tester_sw is set to NoneType. This patch fixes the problem. Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py | 35 --- 1 file changed, 20 insertions(+), 15 deletions(-) diff

Re: [Ryu-devel] ryu-IGMP

2014-03-06 Thread Yuichi Ito
hi. simple_switch_igmp.py requires rewriting according to environments when a querier does not exist. please set the followings: - the datapath id of VM-2 switch - the port number of VM-2 switch that is linked to VM-7 https://github.com/osrg/ryu/blob/master/ryu/app/simple_switch_igmp.py#L38

[Ryu-devel] [PATCH] of14: set default attributes for meter-related classes

2014-03-02 Thread Yuichi Ito
Signed-off-by: Yuichi Ito --- ryu/ofproto/ofproto_v1_4_parser.py | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/ryu/ofproto/ofproto_v1_4_parser.py b/ryu/ofproto/ofproto_v1_4_parser.py index 4dfbb45..ce232e0 100644 --- a/ryu/ofproto/ofproto_v1_4_parser.py +++ b

[Ryu-devel] [PATCH] of13: set default attributes for meter-related classes

2014-03-02 Thread Yuichi Ito
Signed-off-by: Yuichi Ito --- ryu/ofproto/ofproto_v1_3_parser.py | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/ryu/ofproto/ofproto_v1_3_parser.py b/ryu/ofproto/ofproto_v1_3_parser.py index 6c13fe6..7afaf7b 100644 --- a/ryu/ofproto/ofproto_v1_3_parser.py +++ b

Re: [Ryu-devel] About restAPI question for more actions at one flow entry

2014-02-06 Thread Yuichi Ito
Hi, the bug has been fixed in current version. please try Ryu 3.6. thanks. On Fri, 7 Feb 2014 12:16:44 +0800 jalee wrote: > Hi Yuichi, > Do you have fix version can resolve ofctl_rest bug? > Thanks, > > Jalee > > > -Original Message- > From: Yuichi Ito [m

Re: [Ryu-devel] [PATCH] sw test tool: FIX: permit the order different of lists in the case of comparison

2014-02-06 Thread Yuichi Ito
On Fri, 7 Feb 2014 10:04:22 +0900 Simon Horman wrote: > On Fri, Feb 07, 2014 at 09:48:09AM +0900, FUJITA Tomonori wrote: >> On Fri, 07 Feb 2014 09:42:45 +0900 >> Yuichi Ito wrote: >> >>> some switches return the instruction list of flow_mod messages in the &g

[Ryu-devel] [PATCH] sw test tool: correct the comparison method to treat instructions as the set

2014-02-06 Thread Yuichi Ito
according to OF spec, the instruction is treated as the set. this means that the order of instructions which flow_stat message returns is inconstant. this patch corrects the comparison method of instructions by sorting in a particular order before comparison. Signed-off-by: Yuichi Ito --- ryu

Re: [Ryu-devel] [PATCH] sw test tool: FIX: permit the order different of lists in the case of comparison

2014-02-06 Thread Yuichi Ito
2)]), >> OFPInstructionMeter(meter_id=1) >> ] >> >> this patch makes tester.py ignore the order different. >> >> Signed-off-by: Yuichi Ito >> --- >> ryu/tests/switch/tester.py |3 +++ >> 1 file changed, 3 insertions(+) >> >>

Re: [Ryu-devel] [PATCH] sw test tool: FIX: permit the order different of lists in the case of comparison

2014-02-06 Thread Yuichi Ito
I will correct the description and repost later. please do not mind this patch. On Fri, 07 Feb 2014 09:42:45 +0900 Yuichi Ito wrote: > some switches return the instruction list of flow_mod messages in the > different order. > > for example, when a flow entry was instal

Re: [Ryu-devel] [PATCH] sw test tool: FIX: permit the order different of lists in the case of comparison

2014-02-06 Thread Yuichi Ito
On Fri, 07 Feb 2014 09:48:09 +0900 (JST) FUJITA Tomonori wrote: > On Fri, 07 Feb 2014 09:42:45 +0900 > Yuichi Ito wrote: > >> some switches return the instruction list of flow_mod messages in the >> different order. >> >> for example, when a flow entry was i

[Ryu-devel] [PATCH] sw test tool: FIX: return the transaction id when flow entries were deleted

2014-02-06 Thread Yuichi Ito
OfTester._test_initialize() expects the transaction id as a return value of TargetSw.del_test_flow() for checking the error. but this method at current version does not return the transaction id. this patch gets del_test_flow() to return the transaction id. Signed-off-by: Yuichi Ito --- ryu

[Ryu-devel] [PATCH] sw test tool: FIX: permit the order different of lists in the case of comparison

2014-02-06 Thread Yuichi Ito
return the flow entry with the instructions as: [ OFPInstructionActions(actions=[OFPActionOutput(port=2)]), OFPInstructionMeter(meter_id=1) ] this patch makes tester.py ignore the order different. Signed-off-by: Yuichi Ito --- ryu/tests/switch/tester.py |3 +++ 1 file changed, 3

[Ryu-devel] the newest ryu does not operate with applications

2014-02-05 Thread Yuichi Ito
hi. I encountered the following errors by the newest ryu. > Traceback (most recent call last): > File "/usr/local/bin/ryu-manager", line 9, in > load_entry_point('ryu==3.6', 'console_scripts', 'ryu-manager')() > File "/usr/local/lib/python2.7/dist-packages/ryu/cmd/manager.py", line 73,

Re: [Ryu-devel] RYU controller enhancements required

2014-02-03 Thread Yuichi Ito
hi. the interface of ofctl_rest treats instructions and actions similarly. please try this: curl -X POST -d '{"dpid": 1, "match": {"in_port": 9900, "dl_vlan": 100}, "actions": [{"type": "METER", "meter_id": 1},

[Ryu-devel] [PATCH] ofctl_v1_2/3: fix SET_FIELD action and some match fields

2014-01-22 Thread Yuichi Ito
asses: * eth_dst, eth_src, vlan_vid, mpls_label - "arp_spa" and "arp_tpa" were not displayed as IPv4 address Signed-off-by: Yuichi Ito --- ryu/lib/ofctl_v1_2.py | 21 ++--- ryu/lib/ofctl_v1_3.py | 22 +++--- 2 files changed, 37 insertions(

Re: [Ryu-devel] About restAPI question for more actions at one flow entry

2014-01-22 Thread Yuichi Ito
rsion ofctl_rest. please wait for correction. thanks. On Wed, 22 Jan 2014 19:19:10 +0800 jalee wrote: > Hi Yuichi, > So, ryu controller just do push mpls ethertype right? the set-field actions > can we do that? > Such as: > If I want to push new mpls label "777",

Re: [Ryu-devel] About restAPI question for more actions at one flow entry

2014-01-22 Thread Yuichi Ito
1fff1021 or 131073 > > Such as: > > > > "actions": [{"type": "OUTPUT", "port": "8"}, > > {"type": "PUSH_MPLS", "ethertype": > "884720001fff102

Re: [Ryu-devel] About restAPI question for more actions at one flow entry

2014-01-21 Thread Yuichi Ito
Hi. > curl -d > '{"cookie":"1","dpid":"0049","priority":"101","match":{"in_port":"7","dl_src":"00901a400011","dl_dst":"00909941","dl_vlan":"101","dl_vlan_pcp":"0","dl_type":"2048","nw_tos":"0","nw_proto":"17","nw_src":"30.0.0.1","nw_dst":"40.0.0.1","tp_src":"2001","tp_dst":"2001"}

Re: [Ryu-devel] About restAPI question

2014-01-19 Thread Yuichi Ito
/dist-packages/ryu/app$ > ryu-manager ofctl_rest.py > > loading app ofctl_rest.py > > loading app ryu.controller.ofp_handler > > instantiating app None of DPSet > > creating context dpset > > creating context wsgi > > instantiating app ryu.controller.ofp_

Re: [Ryu-devel] About restAPI question

2014-01-19 Thread Yuichi Ito
Hi. > lab@lab-virtual-machine:~/ryu/ryu/lib$ curl -d '{"dpid": "0031", > "priority":"32768", "match":{"in_port":"1"}, "actions":[{"port":"3"}]}' > http://127.0.0.1:8080/stats/flowentry/add please try with the action type such as: curl -d '{"dpid": "0031", "priority":"32

[Ryu-devel] [PATCH] test tool: run_mininet: support CPqD Software Switch

2014-01-16 Thread Yuichi Ito
-by: Yuichi Ito --- ryu/tests/switch/run_mininet.py | 27 --- 1 file changed, 24 insertions(+), 3 deletions(-) diff --git a/ryu/tests/switch/run_mininet.py b/ryu/tests/switch/run_mininet.py index 40ef399..03bb91d 100755 --- a/ryu/tests/switch/run_mininet.py +++ b/ryu

[Ryu-devel] [PATCH] rest_firewall: support IPv6 match conditions

2014-01-14 Thread Yuichi Ito
uot;: "IPv6", "ipv6_src": "10::/64", "rule_id": 1, "actions": "ALLOW" } ] } ], "switch_id": "0001" } ] Signed-off-by: Yuichi Ito --- ryu/ap

[Ryu-devel] [PATCH] ofctl_v1_2/3: fix match conditions about ARP

2014-01-13 Thread Yuichi Ito
src=192.168.0.0/24 actions=output:2 curl -X POST -d '{"dpid": 1, "match": {"dl_type": 2054, "nw_src": "192.168.0.0/24"}, "actions": [{"type

  1   2   3   4   >