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] REST_Router API

2014-05-21 Thread Zubair Hafeez
Hello , I am unable to set the address for router using REST API as mentioned in Ryu frame work. Set the addresses “172.16.20.1/24” and “172.16.30.30/24” for router s1. i am using the following on the controller :curl -X POST -d ’{"address":"172.16.20.1/24"}’ http://localhost:8080/router/000

[Ryu-devel] A question about import ryu.topology.api

2014-05-21 Thread Karthik Sharma
I have a added import ryu.topology.api to my simple_switch.py and I am running it as bin/ryu-manager ryu/app/simple_switch.py According to my understanding, It runs ryu.topology.switches.Switches because Switches is the class in that python module (ryu.topology) that inherits app_manager.RyuApp.

[Ryu-devel] [PATCH] rest_topology: use decorator api

2014-05-21 Thread Satoshi Kobayashi
Signed-off-by: Satoshi Kobayashi --- ryu/app/rest_topology.py | 74 +- 1 files changed, 34 insertions(+), 40 deletions(-) diff --git a/ryu/app/rest_topology.py b/ryu/app/rest_topology.py index 82a1755..5eed243 100644 --- a/ryu/app/rest_topology.py +

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

2014-05-21 Thread FUJITA Tomonori
On Wed, 21 May 2014 17:19:30 +0900 Yuichi Ito wrote: > 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 t

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

2014-05-21 Thread Yuichi Ito
E:1026, 8: Instance of 'DummyDatapath' has no 'set_xid' member (but some types could not be inferred) (maybe-no-member) E:1027, 8: Instance of 'DummyDatapath' has no 'send_msg' member (but some types could not be inferred) (maybe-no-member) NOTE: These methods are never called. This patch is onl

[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 inde

[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/tester

[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..56b

[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 +49

[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: