[Ryu-devel] [PATCH 8/8] BGPSpeaker/net_ctrl: Support multiple RPC sessions

2016-12-15 Thread IWASE Yusuke
Signed-off-by: IWASE Yusuke --- ryu/services/protocols/bgp/net_ctrl.py | 36 ++ 1 file changed, 24 insertions(+), 12 deletions(-) diff --git a/ryu/services/protocols/bgp/net_ctrl.py b/ryu/services/protocols/bgp/net_ctrl.py index 400032c..972d0b9 100644 --- a/ryu/

[Ryu-devel] [PATCH 2/8] rpc_cli: Close peer connections when exiting prompt

2016-12-15 Thread IWASE Yusuke
Signed-off-by: IWASE Yusuke --- ryu/cmd/rpc_cli.py | 33 ++--- 1 file changed, 26 insertions(+), 7 deletions(-) diff --git a/ryu/cmd/rpc_cli.py b/ryu/cmd/rpc_cli.py index 2519c8b..2c1eb6b 100755 --- a/ryu/cmd/rpc_cli.py +++ b/ryu/cmd/rpc_cli.py @@ -56,16 +56,18 @@ cla

[Ryu-devel] [PATCH 6/8] BGPSpeaker/net_ctrl: Move _send_*_response() function

2016-12-15 Thread IWASE Yusuke
This patch moves _send_*_response() function into RpcSession class to use socket wrapper methods for maintaining the connection state safely. Signed-off-by: IWASE Yusuke --- ryu/services/protocols/bgp/net_ctrl.py | 27 --- 1 file changed, 12 insertions(+), 15 deletions(-)

[Ryu-devel] [PATCH 7/8] BGPSpeaker/base: Stop child activity by name

2016-12-15 Thread IWASE Yusuke
This patch enables Activity base to stop the child activity by name. Signed-off-by: IWASE Yusuke --- ryu/services/protocols/bgp/base.py | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/ryu/services/protocols/bgp/base.py b/ryu/services/protocols/bgp/base.py index dd658a

[Ryu-devel] [PATCH 0/8] rpc_cli: Enable to execute a single command

2016-12-15 Thread IWASE Yusuke
The following patches enable 'ryu rpc-cli' command to execute a single command not interactively. e.g.) $ ryu rpc-cli --peers=bgp=localhost:50002 \ --command='request bgp operator.show [{"params": ["rib", "ipv4"]}]' This feature is convenient to controle BGPSpeaker states via shell commands, fo

[Ryu-devel] [PATCH 1/8] rpc_cli: Enable to execute a single command

2016-12-15 Thread IWASE Yusuke
This patch enables 'ryu rpc-cli' command to execute a single command not interactively. e.g.) $ ryu rpc-cli --peers=bgp=localhost:50002 \ --command='request bgp operator.show [{"params": ["rib", "ipv4"]}]' This feature is convenient to controle BGPSpeaker states via shell commands, for example.

[Ryu-devel] [PATCH 3/8] rpc_cli: Avoid to use eval()

2016-12-15 Thread IWASE Yusuke
Signed-off-by: IWASE Yusuke --- ryu/cmd/rpc_cli.py | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/ryu/cmd/rpc_cli.py b/ryu/cmd/rpc_cli.py index 2c1eb6b..25b92fa 100755 --- a/ryu/cmd/rpc_cli.py +++ b/ryu/cmd/rpc_cli.py @@ -31,6 +31,7 @@ from __future__ import print_f

[Ryu-devel] [PATCH 4/8] BGPSpeaker/net_ctrl: Pylint and fixes of typos

2016-12-15 Thread IWASE Yusuke
Signed-off-by: IWASE Yusuke --- ryu/services/protocols/bgp/api/rpc_log_handler.py | 4 +- ryu/services/protocols/bgp/net_ctrl.py| 68 +++ 2 files changed, 35 insertions(+), 37 deletions(-) diff --git a/ryu/services/protocols/bgp/api/rpc_log_handler.py b/ryu/serv

[Ryu-devel] [PATCH 5/8] BGPSpeaker/net_ctrl: Close RPC session when disconnected

2016-12-15 Thread IWASE Yusuke
This patch fixes net_ctrl to check whether socket is connected and to close RPC session when disconnected by RPC peer. Signed-off-by: IWASE Yusuke --- ryu/services/protocols/bgp/net_ctrl.py | 30 +- 1 file changed, 17 insertions(+), 13 deletions(-) diff --git a/ryu/s

[Ryu-devel] dhcp server with ryu controller

2016-12-15 Thread Munther Numan
Greeting , Dear all, I just would like to ask you if I need to add dhcp server in my experiment, is any example code for it or there is already code for dhcp server with Ryu controller? Best Regards Munther Numan Master Student Faculty of Engineering University Putra Malaysia -

Re: [Ryu-devel] Can't see MultipartReply messages, even though they exist.

2016-12-15 Thread Iwase Yusuke
Hi, I guess there some syntax errors in your switch_multipart_reply_handler(). Here is my code which works well on my environment. $ git diff diff --git a/ryu/app/simple_switch_13.py b/ryu/app/simple_switch_13.py index 3e7c598..7d08658 100644 --- a/ryu/app/simple_switch_13.py +++ b/ryu/app/simp

[Ryu-devel] [PATCH] rest_router: Fix ARP THA in reply message

2016-12-15 Thread IWASE Yusuke
Currently, rest_router sends ARP reply messages composing own MAC address in Target MAC Address, and both Sender/Target MAC address are MAC address of rest_router. So, with this reply messages, Wireshark will report "Duplicate IP address detected". This patch fixes this problem. This problem was r

[Ryu-devel] ?????? ryu rest_router.py bug patch

2016-12-15 Thread .......
Hello, I am TICOMM Information Technology Co. Ltd,Shenzhen, China OK, but we hope you will be like last time, e-mail is our company to help broadcast the patch patch. -- -- ??: "IWASE Yusuke";; : 2016??12??16??(??) 9:01 ??:

[Ryu-devel] ?????? ryu rest_router.py bug patch

2016-12-15 Thread .......
OK, but we hope you will be like last time, e-mail is our company to help broadcast the patch patch. -- -- ??: "IWASE Yusuke";; : 2016??12??16??(??) 9:01 ??: "..."<1274096...@qq.com>; : "ryu-devel"; : Re: [Ryu-d

Re: [Ryu-devel] ryu rest_router.py bug patch

2016-12-15 Thread Iwase Yusuke
Hi, Thank you for your reporting again! Could you share us how to solve this problem? Thanks, Iwase On 2016年12月15日 18:03, ... wrote: > Hello, I am TICOMM Information Technology Co. Ltd,Shenzhen, China > > > We use the rest_router.py file to find the bug message on the ARP。 > > > We us

Re: [Ryu-devel] Can't see MultipartReply messages, even though they exist.

2016-12-15 Thread helcio wagner
Hi, Iwase. Thanks for answering. Actually, I've explicitly added the types of MultipartReply messages as the code you shown. Unfortunately, I still don't see any of them (even though they exist). The trace below shows what I'm talking about: --- helcio@ojuara:~/ryu/bin$ ./ryu-manager --verbo