Re: [Ryu-devel] RYU development

2014-06-29 Thread YAMADA Hideki
Hi, (2014/06/30 3:04), Digambar Patil wrote: > Hi, > > Thanks for your reply. > > Yes I am following all the mails, as suggested I am looking at the given > link & BGP & reading the blueprint also. I'll start contribution on the > blueprints related to the RYU, > > RYU - I am reading all the st

Re: [Ryu-devel] XinUI, a Ryu-compatible web-based GUI

2014-06-27 Thread YAMADA Hideki
Great! I've posted similar GUI ryu app. https://github.com/osrg/ryu/tree/master/ryu/app/gui_topology The views of XinUI seems better than gui_topology. So It would be nice if XinUI becomes ryu app or the views are ported to gui_topology. (2014/06/23 23:03), Wei-Lin Chen wrote: > Hello all, > >

[Ryu-devel] [PATCH] document: ryu.app.gui_topology

2014-06-22 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- doc/source/configuration.rst |1 + doc/source/gui.png | Bin 0 -> 24672 bytes doc/source/gui.rst | 33 + 3 files changed, 34 insertions(+) create mode 100644 doc/source/gui.png create mode 100644

Re: [Ryu-devel] [PATCH v2 0/3] Add GUI support

2014-06-17 Thread YAMADA Hideki
(2014/06/18 10:11), FUJITA Tomonori wrote: > On Tue, 17 Jun 2014 13:42:20 +0900 > YAMADA Hideki wrote: > >> This series provide new GUI. >> >> Change of v2: >> * fix pep8 >> >> The advantages to previous (1 year ago) patch are: >> * Integrated

Re: [Ryu-devel] [PATCH v2 3/3] Add GUI app

2014-06-16 Thread YAMADA Hideki
(2014/06/17 13:54), YAMAMOTO Takashi wrote: >> +3. Access http://:8080 with your web browser. > > root directory? > Yes. I think it is better than typing such as "http://IPADDRESS:8080/topology/"; into address bar. But either is fine to me. > YAMAMOTO Takashi > --

[Ryu-devel] [PATCH v2 2/3] ws_topology: Handle WebSocket disconnection

2014-06-16 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/app/ws_topology.py |4 1 file changed, 4 insertions(+) diff --git a/ryu/app/ws_topology.py b/ryu/app/ws_topology.py index d2f4257..7840da0 100644 --- a/ryu/app/ws_topology.py +++ b/ryu/app/ws_topology.py @@ -34,6 +34,7 @@ $ sudo mn --controller

[Ryu-devel] [PATCH v2 3/3] Add GUI app

2014-06-16 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/app/gui_topology/gui_topology.py | 68 ryu/app/gui_topology/html/index.html | 12 ++ ryu/app/gui_topology/html/router.svg | 25 +++ ryu/app/gui_topology/html/ryu.topology.css | 30 ryu/app/gui_topology/html

[Ryu-devel] [PATCH v2 1/3] ws_topology: Avoid controller name confliction

2014-06-16 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/app/ws_topology.py |7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/ryu/app/ws_topology.py b/ryu/app/ws_topology.py index 5679c9b..d2f4257 100644 --- a/ryu/app/ws_topology.py +++ b/ryu/app/ws_topology.py @@ -59,7 +59,7 @@ class

[Ryu-devel] [PATCH v2 0/3] Add GUI support

2014-06-16 Thread YAMADA Hideki
This series provide new GUI. Change of v2: * fix pep8 The advantages to previous (1 year ago) patch are: * Integrated as ryu app (no need other server) * More simple code YAMADA Hideki (3): ws_topology: Avoid controller name confliction ws_topology: Handle WebSocket disconnection Add GUI

[Ryu-devel] [PATCH 0/3] Add GUI support

2014-06-16 Thread YAMADA Hideki
This series provide new GUI. The advantages to previous (1 year ago) patch are: * Integrated as ryu app (no need other server) * More simple code YAMADA Hideki (3): ws_topology: Avoid controller name confliction ws_topology: Handle WebSocket disconnection Add GUI app ryu/app/gui_topology

[Ryu-devel] [PATCH 1/3] ws_topology: Avoid controller name confliction

2014-06-16 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/app/ws_topology.py |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ryu/app/ws_topology.py b/ryu/app/ws_topology.py index 5679c9b..ee63bf8 100644 --- a/ryu/app/ws_topology.py +++ b/ryu/app/ws_topology.py @@ -59,7 +59,7 @@ class

[Ryu-devel] [PATCH 2/3] ws_topology: Handle WebSocket disconnection

2014-06-16 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/app/ws_topology.py |4 1 file changed, 4 insertions(+) diff --git a/ryu/app/ws_topology.py b/ryu/app/ws_topology.py index ee63bf8..fadf25f 100644 --- a/ryu/app/ws_topology.py +++ b/ryu/app/ws_topology.py @@ -34,6 +34,7 @@ $ sudo mn --controller

[Ryu-devel] [PATCH 3/3] Add GUI app

2014-06-16 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/app/gui_topology/gui_topology.py | 67 ryu/app/gui_topology/html/index.html | 12 ++ ryu/app/gui_topology/html/router.svg | 25 +++ ryu/app/gui_topology/html/ryu.topology.css | 30 ryu/app/gui_topology/html

[Ryu-devel] [PATCH v2] app_manager: Let application module to use require_app()

2014-06-13 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/app/ofctl/api.py |2 +- ryu/base/app_manager.py| 16 +++- ryu/services/protocols/vrrp/api.py |2 +- ryu/topology/api.py|2 +- 4 files changed, 14 insertions(+), 8 deletions(-) diff --git a

Re: [Ryu-devel] [PATCH] app_manager: Let client app to use require_app() directly

2014-06-12 Thread YAMADA Hideki
(2014/06/11 16:56), YAMADA Hideki wrote: > (2014/06/11 16:32), FUJITA Tomonori wrote: >> On Thu, 5 Jun 2014 19:08:48 +0900 >> YAMADA Hideki wrote: >> >>> >>> Signed-off-by: YAMADA Hideki >>> --- >>>ryu/base/app_manager.py | 14

Re: [Ryu-devel] [PATCH] app_manager: Let client app to use require_app() directly

2014-06-11 Thread YAMADA Hideki
(2014/06/11 16:32), FUJITA Tomonori wrote: > On Thu, 5 Jun 2014 19:08:48 +0900 > YAMADA Hideki wrote: > >> >> Signed-off-by: YAMADA Hideki >> --- >> ryu/base/app_manager.py | 14 +++--- >> 1 file changed, 7 insertions(+), 7 deletions(-) >&

[Ryu-devel] [PATCH] app_manager: Let client app to use require_app() directly

2014-06-05 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/base/app_manager.py | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/ryu/base/app_manager.py b/ryu/base/app_manager.py index 25b043a..59a8b4b 100644 --- a/ryu/base/app_manager.py +++ b/ryu/base/app_manager.py @@ -68,14

Re: [Ryu-devel] [PATCH] fix failure of instantiating app

2014-06-03 Thread YAMADA Hideki
I have found same problem when I use rest_topology and ws_topology. And this patch fixed it. (2014/05/29 13:40), Yoshihiro Kaneko wrote: > For example, app-A has app-X in CONTEXTS, and app-B depends on app-X as > SERVICE. When app-B is specified in the app-lists before app-A, the > instantiating

Re: [Ryu-devel] EventOFPPacketIn in OF1.2

2014-05-23 Thread YAMADA Hideki
Hi, (2014/05/23 11:45), Qun Nguyn Thanh wrote: > Dear Ryu-Devel, > > I would like to ask one question about my issues. > > When controller send "Echo Reply" to switch. I want to do something with > this function. So I use > > > *@set_ev_cls(ofp_event.EventOFPPacketIn, [HANDSHAKE_DISPATCHER, > CONF

Re: [Ryu-devel] [PATCH] wsgi: add ws_topology application

2014-05-22 Thread YAMADA Hideki
(2014/05/22 20:23), Satoshi Kobayashi wrote: > - Topology change is notified >- JSON-RPC/WebSocket > > Signed-off-by: Satoshi Kobayashi > --- > ryu/app/ws_topology.py | 105 > > ryu/app/wsgi.py| 34 +++- > 2 files ch

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

2014-05-22 Thread YAMADA Hideki
(2014/05/21 18:12), Satoshi Kobayashi wrote: > > 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.p

Re: [Ryu-devel] Can't trace the reason for the message (switch Switch Port)

2014-05-20 Thread YAMADA Hideki
Hi, It might be this line. https://github.com/karthik82/ryu/blob/master/ryu/topology/switches.py#L492 Because your app import ryu.topology.api, ryu.topology.switches is loaded automatically. (2014/05/21 12:10), Karthik Sharma wrote: > Hi > > When I run my modified version of simple_switch.py I

[Ryu-devel] [PATCH 2/4] Add WebSocket JSON RPC feature

2014-05-19 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/app/wsgi.py | 44 1 file changed, 44 insertions(+) diff --git a/ryu/app/wsgi.py b/ryu/app/wsgi.py index 406fbab..6870731 100644 --- a/ryu/app/wsgi.py +++ b/ryu/app/wsgi.py @@ -23,6 +23,11 @@ from ryu import cfg

[Ryu-devel] [PATCH 1/4] simple_switch_websocket_13: fix typo

2014-05-19 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/app/simple_switch_websocket_13.py |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ryu/app/simple_switch_websocket_13.py b/ryu/app/simple_switch_websocket_13.py index a6fd989..2cee238 100644 --- a/ryu/app/simple_switch_websocket_13

[Ryu-devel] [PATCH 4/4] simple_switch_websocket_13: Add get_arp_table RPC method

2014-05-19 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/app/simple_switch_websocket_13.py | 31 ++- 1 file changed, 26 insertions(+), 5 deletions(-) diff --git a/ryu/app/simple_switch_websocket_13.py b/ryu/app/simple_switch_websocket_13.py index 2cee238..3168d5b 100644 --- a/ryu

[Ryu-devel] [PATCH 3/4] pip-requires: WebSocket-RPC requires tinyrpc

2014-05-19 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- tools/pip-requires |1 + 1 file changed, 1 insertion(+) diff --git a/tools/pip-requires b/tools/pip-requires index 2528ebc..4ed8bc7 100644 --- a/tools/pip-requires +++ b/tools/pip-requires @@ -7,3 +7,4 @@ paramiko routes six>=1.4.0 webob>=1.0.8 +t

[Ryu-devel] [PATCH 0/4] Support JSON-RPC 2.0 over WebSocket

2014-05-19 Thread YAMADA Hideki
YAMADA Hideki (4): simple_switch_websocket_13: fix typo Add WebSocket JSON RPC feature pip-requires: WebSocket-RPC requires tinyrpc simple_switch_websocket_13: Add get_arp_table RPC method ryu/app/simple_switch_websocket_13.py | 35 -- ryu/app/wsgi.py

Re: [Ryu-devel] [PATCH 4/4] Add PacketIn dumper websocket application

2014-05-18 Thread YAMADA Hideki
(2014/05/19 11:03), Satoshi Kobayashi wrote: > 2014-05-16 5:21 GMT+09:00 FUJITA Tomonori : > >> On Thu, 15 May 2014 13:30:53 +0900 >> Satoshi Kobayashi wrote: >> >> +class SimpleSwitchWebSocketController(ControllerBase): >> +def __init__(self, req, link, data, **config): >> +

Re: [Ryu-devel] [PATCH 4/4] Add PacketIn dumper websocket application

2014-05-14 Thread YAMADA Hideki
Hi Kobayashi-san, Thank you for review. (2014/05/15 11:20), Satoshi Kobayashi wrote: > Hi YAMADA-san, > > 2014-05-14 19:11 GMT+09:00 YAMADA Hideki : > >> >> Signed-off-by: YAMADA Hideki >> --- >> ryu/app/simple_switch_websocket_13.py | 91 >> +++

[Ryu-devel] [PATCH 4/4] Add PacketIn dumper websocket application

2014-05-14 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/app/simple_switch_websocket_13.py | 91 + 1 file changed, 91 insertions(+) create mode 100644 ryu/app/simple_switch_websocket_13.py diff --git a/ryu/app/simple_switch_websocket_13.py b/ryu/app/simple_switch_websocket_13

[Ryu-devel] [PATCH 0/4] WebSocket support

2014-05-14 Thread YAMADA Hideki
This patch set add WebSocket support. This is based on the discussion of following thread. http://thread.gmane.org/gmane.network.ryu.devel/5814 YAMADA Hideki (4): hub: import semaphore Import eventlet.websocket from master branch wsgi: support WebSocket Add PacketIn dumper websocket

[Ryu-devel] [PATCH 3/4] wsgi: Support WebSocket

2014-05-14 Thread YAMADA Hideki
- Hack webob.dec.wsgify for taking start_response function. - Add websocket handshake helper method to controller. Signed-off-by: YAMADA Hideki --- ryu/app/wsgi.py | 15 +-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/ryu/app/wsgi.py b/ryu/app/wsgi.py index

[Ryu-devel] [PATCH 1/4] hub: import semaphore

2014-05-14 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/lib/hub.py |3 +++ 1 file changed, 3 insertions(+) diff --git a/ryu/lib/hub.py b/ryu/lib/hub.py index 95199e7..c68f19b 100644 --- a/ryu/lib/hub.py +++ b/ryu/lib/hub.py @@ -29,6 +29,7 @@ if HUB_TYPE == 'eventlet': import eventlet

[Ryu-devel] [PATCH 2/4] Import eventlet.websocket from master branch

2014-05-14 Thread YAMADA Hideki
Because eventlet 0.14 doen't support RFC 6455. Signed-off-by: YAMADA Hideki --- ryu/contrib/_eventlet/websocket.py | 656 ryu/lib/hub.py |3 + 2 files changed, 659 insertions(+) create mode 100644 ryu/contrib/_eventlet/__init

Re: [Ryu-devel] Topology representation in RYU controller

2014-04-22 Thread YAMADA Hideki
Hi, (2014/04/22 11:29), Karthik Sharma wrote: > This is the code for class Switches from > https://github.com/osrg/ryu/blob/master/ryu/topology/switches.py#L429 > > The member variables of particular interest to me in the class Switches are > the following. > > self.dps = {}

[Ryu-devel] [PATCH] tests/integrated: fix mac address format

2013-09-25 Thread YAMADA Hideki
er: self.value = addr ryu-manager: File "/home/hideki/git/ryu/.venv/local/lib/python2.7/site-packages/netaddr/eui/__init__.py", line 434, in _set_value ryu-manager: % (value, self._module.version)) ryu-manager: AddrFormatError: address '\xff\xff\xff\xff\xff\xff' is not an E

Re: [Ryu-devel] [PATCH] tests/integrated: fix mac address format

2013-09-25 Thread YAMADA Hideki
OK. I'll resend the patch. (2013/09/26 4:29), FUJITA Tomonori wrote: > On Wed, 25 Sep 2013 18:27:06 +0900 > YAMADA Hideki wrote: > >> This fix following error. >> > > Thanks, can you fix the following pep8 warning and resend? > > ryu/tests/integrated/

[Ryu-devel] [PATCH] tests/integrated: fix mac address format

2013-09-25 Thread YAMADA Hideki
er: self.value = addr ryu-manager: File "/home/hideki/git/ryu/.venv/local/lib/python2.7/site-packages/netaddr/eui/__init__.py", line 434, in _set_value ryu-manager: % (value, self._module.version)) ryu-manager: AddrFormatError: address '\xff\xff\xff\xff\xff\xff' is not an

Re: [Ryu-devel] Ryu GUI

2013-07-30 Thread YAMADA Hideki
Hi, The GUI document is available at my github wiki. https://github.com/yamada-h/ryu/wiki/GUI (2013/07/31 12:55), Thomas Perniciaro wrote: > Has anyone got full instructions on how to install the GUI. I'm new to Ryu > and have installed the latest GUI patch from git and have included the path > f

Re: [Ryu-devel] run_tests_with_ovs12.py failed for No such file or director

2013-07-12 Thread YAMADA Hideki
Hi, Maybe there is no venv. Try "./run_tests.sh -i" instead of direct execution. It will create venv automatically. (2013/07/12 17:13), xiaohang wrote: > Hi,all > > > When I run the test run_tests_with_ovs12.py, get the below error: > could you tell me how to modify this error? > > > and ,are

[Ryu-devel] [PATCH 2/2] tests/unit/packet/test_vrrp: add tests for is_valid()

2013-05-20 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/tests/unit/packet/test_vrrp.py | 92 +++- 1 files changed, 91 insertions(+), 1 deletions(-) diff --git a/ryu/tests/unit/packet/test_vrrp.py b/ryu/tests/unit/packet/test_vrrp.py index 32dbdb4..282c9fa 100644 --- a/ryu/tests

[Ryu-devel] [PATCH 1/2] lib/packet/vrrp: fix is_valid()

2013-05-20 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/lib/packet/vrrp.py | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/ryu/lib/packet/vrrp.py b/ryu/lib/packet/vrrp.py index 7c50fed..8b6140a 100644 --- a/ryu/lib/packet/vrrp.py +++ b/ryu/lib/packet/vrrp.py @@ -395,7 +395,7

[Ryu-devel] [PATCH] test_parser_v10: remove tests for __str__

2013-05-20 Thread YAMADA Hideki
tr(self.property['val'])) KeyError: 'property' == ERROR: test_parser (ryu.tests.unit.ofproto.test_parser_v10.TestOFPSwitchFeatures) ------ Traceback (most recent call last): File "/home/hidek

Re: [Ryu-devel] repo of vrrp support (patch v3)

2013-05-15 Thread YAMADA Hideki
00 2001 > Message-Id: > <635ecac005288193204321a6778b8fe6cadd9026.1368610234.git.yamah...@valinux.co.jp> > From: Isaku Yamahata > Date: Wed, 15 May 2013 18:09:49 +0900 > Subject: [PATCH 1/1] base/app_manager: context might be RyuApp > > The class used for context might be RyuApp. So methods of RyuApp needs

Re: [Ryu-devel] repo of vrrp support (patch v3)

2013-05-14 Thread YAMADA Hideki
Hi, The tree doesn't pass integrated tests. Event-loop thread of context instance doesn't seem to start. https://github.com/yamahata/ryu/commit/16fe17b84f27ef19486c290b9ff0b22b6699ded6 diff --git a/ryu/base/app_manager.py b/ryu/base/app_manager.py index 1250299..6f2b54d 100644 --- a/ryu/base/app

[Ryu-devel] [PATCH v3 6/6] run_tests.sh: support GUI auto testing

2013-05-13 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- run_tests.sh | 34 + ryu/tests/gui/run_servers.sh | 43 ++ 2 files changed, 77 insertions(+), 0 deletions(-) create mode 100755 ryu/tests/gui/run_servers.sh diff --git

[Ryu-devel] [PATCH v3 4/6] add test for GUI

2013-05-13 Thread YAMADA Hideki
run_tests_with_XXX.py: executable test code mn_ctl.py: mininet controller server Signed-off-by: YAMADA Hideki --- ryu/tests/gui/lib/elements.py | 326 + ryu/tests/gui/lib/test_gui.py | 578 +++ ryu/tests/gui

[Ryu-devel] [PATCH v3 5/6] test-requires: selenium for GUI test

2013-05-13 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- tools/test-requires |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/tools/test-requires b/tools/test-requires index da50e3e..a2f0a4e 100644 --- a/tools/test-requires +++ b/tools/test-requires @@ -2,3 +2,4 @@ coverage nose pep8 pylint

[Ryu-devel] [PATCH v3 2/6] GUI: import perfect-scrollbar.js

2013-05-13 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/gui/static/js/contrib/jquery.mousewheel.js | 84 +++ ryu/gui/static/js/contrib/perfect-scrollbar.js | 313 2 files changed, 397 insertions(+), 0 deletions(-) create mode 100644 ryu/gui/static/js/contrib/jquery.mousewheel.js

[Ryu-devel] [PATCH v3 3/6] pip-requires: flask, gevent-websocket for GUI

2013-05-13 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- tools/pip-requires |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/tools/pip-requires b/tools/pip-requires index 208b9b8..93f18d4 100644 --- a/tools/pip-requires +++ b/tools/pip-requires @@ -3,3 +3,5 @@ routes webob>=1.0.8 param

[Ryu-devel] [PATCH v3 0/6] GUI support

2013-05-13 Thread YAMADA Hideki
using Selenium YAMADA Hideki (6): add GUI modules GUI: import perfect-scrollbar.js pip-requires: flask, gevent-websocket for GUI add test for GUI test-requires: selenium for GUI test run_tests.sh: support GUI auto testing run_tests.sh | 34 + r

[Ryu-devel] [PATCH 3/3] run_tests.sh: support integrated tests

2013-04-25 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- run_tests.sh | 14 ++ 1 files changed, 14 insertions(+), 0 deletions(-) diff --git a/run_tests.sh b/run_tests.sh index 56609d2..bf703a6 100755 --- a/run_tests.sh +++ b/run_tests.sh @@ -11,6 +11,7 @@ usage() { echo " -p, -

[Ryu-devel] [PATCH 2/3] tests/integrated: auto testing script using Mininet and OVS

2013-04-25 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/tests/integrated/run_tests_with_ovs12.py | 108 ++ 1 files changed, 108 insertions(+), 0 deletions(-) create mode 100755 ryu/tests/integrated/run_tests_with_ovs12.py diff --git a/ryu/tests/integrated/run_tests_with_ovs12.py b/ryu

[Ryu-devel] [PATCH 1/3] tests/integrated: Skip MPLS related test

2013-04-25 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/tests/integrated/test_add_flow_v12_actions.py |9 + ryu/tests/integrated/test_add_flow_v12_matches.py |2 ++ 2 files changed, 11 insertions(+), 0 deletions(-) diff --git a/ryu/tests/integrated/test_add_flow_v12_actions.py b/ryu/tests

[Ryu-devel] [PATCH v2 4/5] tests/topology: add test for GUI

2013-04-25 Thread YAMADA Hideki
mn_ctl.py: mininet controller auto_topo.py: auto topology create/delete script gui_client: auto test for GUI (use selenium). Signed-off-by: YAMADA Hideki --- ryu/tests/topology/auto_topo.py | 204 + ryu/tests/topology/gui_client/gui_elements.py | 323 ++ ryu

[Ryu-devel] [PATCH v2 5/5] test-requires: selenium for GUI test

2013-04-25 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- tools/test-requires |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/tools/test-requires b/tools/test-requires index 6f21bff..7ca00ec 100644 --- a/tools/test-requires +++ b/tools/test-requires @@ -3,3 +3,4 @@ nose pep8 pylint==0.25.0

[Ryu-devel] [PATCH v2 3/5] pip-requires: flask, gevent-websocket for GUI

2013-04-25 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- tools/pip-requires |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/tools/pip-requires b/tools/pip-requires index 2a9b4ce..ae42958 100644 --- a/tools/pip-requires +++ b/tools/pip-requires @@ -2,3 +2,5 @@ gevent>=0.13 routes we

[Ryu-devel] [PATCH v2 2/5] GUI: import perfect-scrollbar.js

2013-04-25 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/gui/static/js/contrib/jquery.mousewheel.js | 84 +++ ryu/gui/static/js/contrib/perfect-scrollbar.js | 313 2 files changed, 397 insertions(+), 0 deletions(-) create mode 100644 ryu/gui/static/js/contrib/jquery.mousewheel.js

[Ryu-devel] [PATCH v2 0/5] GUI support

2013-04-25 Thread YAMADA Hideki
ogy/switches.py \ ./ryu/app/rest_topology.py \ ./ryu/app/ofctl_rest.py 2. Run GUI web server $ PYTHONPATH=. ./ryu/gui/controller.py 3. Access to http://127.0.0.1:8000 YAMADA Hideki (5): GUI support GUI: import perfect-scrollbar.js pip-requires: flask, gevent-websocket

Re: [Ryu-devel] topology discovery

2013-04-23 Thread YAMADA Hideki
st-packages/ryu-1.8-py2.7.egg/ryu/base/app_manager.py", > line 40, in register_app > assert not app.name in SERVICE_BRICKS > AssertionError > > Best Regards, > Nikhil > > > On Wed, Apr 24, 2013 at 6:34 AM, YAMADA Hideki > wrote: > >> Hi Nikhil, >> &

Re: [Ryu-devel] Running integrated tests for OF1.2 on custom soft client (not openvswitch)

2013-04-23 Thread YAMADA Hideki
Hi, Perhaps your switch doesn't support ARP_SHA rule. Apply the following patch, and you can skip that test. Regards, Yamada diff --git a/ryu/tests/integrated/test_add_flow_v12_matches.py b/ryu/tests/integrated/test_add_flow_v12_matches.py index 27b3880..9520ae8 100644 --- a/ryu/tests/integrat

Re: [Ryu-devel] topology discovery

2013-04-23 Thread YAMADA Hideki
Hi Nikhil, (2013/04/23 15:23), Nikhil wrote: > Hi, > > > > When i ran ryu topology application with mininet tree topology > > > > ./bin/ryu-manager --verbose --observe-links ryu/topology/switches.py > ryu/topology/dumper.py > > > I am getting below error (exception) > > assert not app.name in SERV

[Ryu-devel] [PATCH 2/4] GUI: import perfect-scrollbar.js

2013-04-15 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/gui/static/js/contrib/jquery.mousewheel.js | 84 +++ ryu/gui/static/js/contrib/perfect-scrollbar.js | 313 2 files changed, 397 insertions(+), 0 deletions(-) create mode 100644 ryu/gui/static/js/contrib/jquery.mousewheel.js

[Ryu-devel] [PATCH 3/4] pip-requires: flask, gevent-websocket for GUI

2013-04-15 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- tools/pip-requires |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/tools/pip-requires b/tools/pip-requires index 2a9b4ce..ae42958 100644 --- a/tools/pip-requires +++ b/tools/pip-requires @@ -2,3 +2,5 @@ gevent>=0.13 routes we

[Ryu-devel] [PATCH 4/4] tests/topology: auto topology create/delete script

2013-04-15 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/tests/topology/auto_topo.py | 204 +++ ryu/tests/topology/mn_ctl.py| 45 + 2 files changed, 249 insertions(+), 0 deletions(-) create mode 100644 ryu/tests/topology/auto_topo.py create mode 100644 ryu/tests

[Ryu-devel] [PATCH 0/4] GUI support

2013-04-15 Thread YAMADA Hideki
/ofctl_rest.py 2. Run GUI web server $ PYTHONPATH=. python ./ryu/gui/controller.py 3. Access to http://127.0.0.1:8000 YAMADA Hideki (4): GUI support GUI: import perfect-scrollbar.js pip-requires: flask, gevent-websocket for GUI tests/topology: auto topology create/delete script ryu/gui

Re: [Ryu-devel] topology discovery on openvswitch using ryu controller.

2013-04-04 Thread YAMADA Hideki
t; - do I have to trigger something inorder for the app to produce an output? > LOG.debug(). Probably it is stdout, with --verbose option of ryu-manager. Please see following link. http://sourceforge.net/mailarchive/forum.php?thread_name=1363256167-4406-1-git-send-email-yamada.hideki%40po.ntts.co.jp&

Re: [Ryu-devel] [PATCH 3/5] topology: Add REST API

2013-04-02 Thread YAMADA Hideki
31, 2013 at 2:51 PM, FUJITA Tomonori < > fujita.tomon...@lab.ntt.co.jp> wrote: > >> On Thu, 28 Mar 2013 18:50:27 +0900 >> YAMADA Hideki wrote: >> >>> 1. run topology/switches and app/rest_topology >>> $ ./bin/ryu-manager --verbose --observe-links ryu/

Re: [Ryu-devel] [PATCH] topology/dumper: fix RyuApp.__init__ signature

2013-03-31 Thread YAMADA Hideki
Thanks. It's my mistake. Acked-by: YAMADA Hideki (2013/04/01 15:13), YAMAMOTO Takashi wrote: > so that this is at least instantiatable. > > Signed-off-by: YAMAMOTO Takashi > --- > ryu/topology/dumper.py | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(

[Ryu-devel] [PATCH] test_parser_v10: fix unittest breakage

2013-03-28 Thread YAMADA Hideki
ibute 'specific_data' -- Ran 1249 tests in 0.991s FAILED (errors=1) Signed-off-by: YAMADA Hideki --- ryu/tests/unit/ofproto/test_parser_v10.py |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/ryu/tests/unit/ofproto/test_parser_v10.py b/ryu/test

Re: [Ryu-devel] REST API for topology module

2013-03-28 Thread YAMADA Hideki
(2013/03/27 15:22), Nikhil wrote: > Thanks a lot. Discovery module is designed to work with only OF1.0. When > can we expect OF1.2 or OF1.3 support? > Sorry, there is no plan for support OF1.2/OF1.3 yet. But I think it is needed. > Best Regards, > Nikhil > > -

[Ryu-devel] [PATCH 5/5] ryu-client: print http response

2013-03-28 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- bin/ryu-client |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/bin/ryu-client b/bin/ryu-client index ea3a3ea..9277964 100755 --- a/bin/ryu-client +++ b/bin/ryu-client @@ -99,7 +99,9 @@ def client_test(): for

[Ryu-devel] [PATCH 4/5] ryu-client: support Topology REST API

2013-03-28 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- bin/ryu-client|5 + ryu/app/client.py | 34 ++ 2 files changed, 39 insertions(+), 0 deletions(-) diff --git a/bin/ryu-client b/bin/ryu-client index 292ee26..ea3a3ea 100755 --- a/bin/ryu-client +++ b/bin/ryu-client

[Ryu-devel] [PATCH 3/5] topology: Add REST API

2013-03-28 Thread YAMADA Hideki
0001"}, {"hw_addr": "1e:7b:0b:fc:0a:2f", "name": "s1-eth2", "port_no": "0002", "dpid": "0001"}], "dpid": "0001"}, {"ports": [{"hw_addr": &q

[Ryu-devel] [PATCH 2/5] topology: exported function returns Switch/Link list instead of event

2013-03-28 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/topology/switches.py |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/ryu/topology/switches.py b/ryu/topology/switches.py index 6b69a00..318473f 100644 --- a/ryu/topology/switches.py +++ b/ryu/topology/switches.py @@ -804,7

[Ryu-devel] [PATCH 1/5] lib/port_no: handling port_no as string

2013-03-28 Thread YAMADA Hideki
Signed-off-by: YAMADA Hideki --- ryu/lib/port_no.py | 31 +++ 1 files changed, 31 insertions(+), 0 deletions(-) create mode 100644 ryu/lib/port_no.py diff --git a/ryu/lib/port_no.py b/ryu/lib/port_no.py new file mode 100644 index 000..33720f3 --- /dev/null

[Ryu-devel] [PATCH 0/5] Topology REST API

2013-03-28 Thread YAMADA Hideki
YAMADA Hideki (5): lib/port_no: handling port_no as string topology: exported function returns Switch/Link list instead of event topology: Add REST API ryu-client: support Topology REST API ryu-client: print http response bin/ryu-client |9 - ryu/app/client.py

Re: [Ryu-devel] REST API for topology module

2013-03-26 Thread YAMADA Hideki
/dumper.py. To run dumper.py, please see following link. http://sourceforge.net/mailarchive/forum.php?thread_name=1363256167-4406-1-git-send-email-yamada.hideki%40po.ntts.co.jp&forum_name=ryu-devel > Best Regards, > Nikhil > > > On Tue, Mar 26, 2013 at 3:27 PM, YAMADA Hideki >

Re: [Ryu-devel] REST API for topology module

2013-03-26 Thread YAMADA Hideki
Hi, Topology module has not REST API yet. I will post REST API patch in this week. (2013/03/26 18:38), Nikhil wrote: > Hello, > > > Does topology module added to ryu controller recently has REST API? If yes, > Please provide me details of the same. > > > Best Regards, > Nikhil > > > > ---

[Ryu-devel] [PATCH 3/3] topology: support link discovery

2013-03-14 Thread YAMADA Hideki
event.py: add link events. switches.py: add link discovery (only of1.0). dumper.py: add handler for link events. TODO: support other OpenFlow version. Signed-off-by: YAMADA Hideki --- bin/ryu-manager |1 + ryu/topology/dumper.py | 79 +- ryu/topology/event.py| 42

[Ryu-devel] [PATCH 2/3] topology: switch discovering module

2013-03-14 Thread YAMADA Hideki
This is an alternative of dpset, which supports event passing. event.py: event class switches.py: switch(datapath) discovery app using ofp_event dumper.py: test and example app using switch discovery event Signed-off-by: YAMADA Hideki --- ryu/topology/dumper.py | 104

[Ryu-devel] [PATCH 1/3] event: Add sync flag and event src for request/reply

2013-03-14 Thread YAMADA Hideki
- To distinguish synchronous request or not. - Event src is needed for asynchronous request too. Signed-off-by: YAMADA Hideki --- ryu/base/app_manager.py |7 ++- ryu/controller/event.py |6 -- 2 files changed, 10 insertions(+), 3 deletions(-) diff --git a/ryu/base

[Ryu-devel] [PATCH 0/3] Topology Discovery

2013-03-14 Thread YAMADA Hideki
EventPortModify> EventLinkDelete to Port> EventLinkDelete to Port> EventPortModify> EventPortModify> YAMADA Hideki (3): Add sync flag and event src for Request/Reply event handling topology: switch discovering module topology: support link discovery bin/ryu-manager |

[Ryu-devel] [PATCH v2 3/3] physicaltopology: switch discovering module

2013-02-22 Thread YAMADA Hideki
This is an alternative of dpset, which supports event passing. event.py: event class switch.py: switch(datapath) discovery app using ofp_event dumper.py: test and example app using switch discovery event TODO: support link discovery using LLDP. Signed-off-by: YAMADA Hideki --- ryu

[Ryu-devel] [PATCH v2 2/3] RyuApp: support app-to-app event passing communication

2013-02-22 Thread YAMADA Hideki
Now, direct method calling is used for app-to-app communication, via _CONTEXTS parameters. This patch is more loose coupling way than direct way. Signed-off-by: YAMADA Hideki --- ryu/base/app_manager.py | 29 + ryu/controller/event.py | 17 + 2

[Ryu-devel] [PATCH v2 1/3] app_manager: allow separated modules of Event and RyuApp

2013-02-22 Thread YAMADA Hideki
Because handler.set_ev_cls() sets observer to module of ev_cls, Event class and RyuApp must be in same module now. This patch let RyuApp have a list of events to be generated in app. So, AppManager can bind sender app and receiver app via event class. Signed-off-by: YAMADA Hideki --- ryu/base

[Ryu-devel] [PATCH v2 0/3] App-to-App event passing

2013-02-22 Thread YAMADA Hideki
ce dispatcher mechanism https://github.com/osrg/ryu/commit/7578e7d6020ea446af4640b17adb0c325c7570a8 YAMADA Hideki (3): app_manager: support separated modules of Event class and RyuApp class RyuApp: support app-to-app event passing communication physicaltopology: switch discovering module

Re: [Ryu-devel] [PATCH] controller: fix that sender thread is never terminated

2013-02-21 Thread YAMADA Hideki
There is some thing wrong. Perhaps line feed code problem. Please apply attached patch. (2013/02/21 17:17), YAMADA Hideki wrote: hi, (2013/02/21 16:17), YAMAMOTO Takashi wrote: hi, Empty send_q.get() is blocking send thread. Because of this issue, datapath state never transit to DEAD

Re: [Ryu-devel] [PATCH] controller: fix that sender thread is never terminated

2013-02-21 Thread YAMADA Hideki
, too. How about following? From 08126332a9d3d59af8e441ecfe78482a90515771 Mon Sep 17 00:00:00 2001 From: YAMADA Hideki Date: Thu, 21 Feb 2013 16:33:47 +0900 Subject: [PATCH] controller: fix that sender thread is never terminated Empty q.get() is blocking send thread. Because of this issue, datapath state never

[Ryu-devel] [PATCH] controller: fix that sender thread is never terminated

2013-02-20 Thread YAMADA Hideki
: YAMADA Hideki --- ryu/controller/controller.py |7 ++- 1 files changed, 6 insertions(+), 1 deletions(-) diff --git a/ryu/controller/controller.py b/ryu/controller/controller.py index 9dc745a..59286ff 100644 --- a/ryu/controller/controller.py +++ b/ryu/controller/controller.py @@ -24,6

Re: [Ryu-devel] [PATCH 2/2] support physical topology discovery

2013-02-19 Thread YAMADA Hideki
>>> +@handler.set_ev_cls(ofp_event.EventOFPPortStatus, >>> handler.MAIN_DISPATCHER) >>> +def port_status_handler(self, ev): >>> +msg = ev.msg >>> +reason = msg.reason >>> +dpid = msg.datapath.id >>> +port_no = msg.desc.port_no >>> +ofproto = msg.data

Re: [Ryu-devel] [PATCH 2/2] support physical topology discovery

2013-02-19 Thread YAMADA Hideki
>> +@handler.set_ev_cls(ofp_event.EventOFPPortStatus, >> handler.MAIN_DISPATCHER) >> +def port_status_handler(self, ev): >> +msg = ev.msg >> +reason = msg.reason >> +dpid = msg.datapath.id > > Do you see msg.datapath.id is None case? > I don't see. Is it possible ca

Re: [Ryu-devel] [PATCH 2/2] support physical topology discovery

2013-02-19 Thread YAMADA Hideki
Hi, (2013/02/20 11:37), Isaku Yamahata wrote: > General comment. Some of logic could be generic, not specific to > datapath tracking. > Thanks for your comment. >> +class DatapathDiscovery(app_manager.RyuApp): >> +_EVENTS = [EventDPEnter, EventDPLeave, >> + EventDPPortAdd, Event

[Ryu-devel] [PATCH 1/2] app_manager: support separated modules of Event class and RyuApp class

2013-02-19 Thread YAMADA Hideki
Because handler.set_ev_cls() sets observer to module of ev_cls, Event class and RyuApp must be in same module. If RyuApp has a list of events to be generated in app, AppManager can bind sender app and receiver app via event class. Signed-off-by: YAMADA Hideki --- ryu/base/app_manager.py |8

[Ryu-devel] [PATCH 2/2] support physical topology discovery

2013-02-19 Thread YAMADA Hideki
This is alternative of dpset. * Don't call other app's method directly. * Use Event request/reply event.py: event class modules datapath.py: datapath discovery app using ofp_event dumper.py: test app using discovery event TODO: support link discovery using LLDP Signed-off-by: YAM

[Ryu-devel] [PATCH 0/2] Improve event mechanism

2013-02-19 Thread YAMADA Hideki
This patch is related with following commit. replace dispatcher mechanism https://github.com/osrg/ryu/commit/7578e7d6020ea446af4640b17adb0c325c7570a8 YAMADA Hideki (2): app_manager: support separated modules of Event class and RyuApp class support physical topology discovery ryu/base

Re: [Ryu-devel] [PATCH v3] controller/ofp_handler: improve version negotiation

2013-02-12 Thread YAMADA Hideki
(2013/02/13 12:06), Isaku Yamahata wrote: > - fix string concatenation. Needs surrounding paren. > - more version checks > - make hello_failed private. (adding "_" prefix) > > Signed-off-by: Isaku Yamahata > --- > ryu/controller/ofp_handler.py | 90 > -

[Ryu-devel] [PATCH] dpset: improve debug message

2013-02-05 Thread YAMADA Hideki
Could not understand "reason" number at a glance. Signed-off-by: YAMADA Hideki --- ryu/controller/dpset.py | 11 +-- 1 files changed, 9 insertions(+), 2 deletions(-) diff --git a/ryu/controller/dpset.py b/ryu/controller/dpset.py index 1660410..32e8594 100644 --- a/ryu/

Re: [Ryu-devel] [PATCH RFC] packet lib: add LLDP support.

2013-02-04 Thread YAMADA Hideki
(2013/02/05 16:17), FUJITA Tomonori wrote: > On Mon, 4 Feb 2013 21:33:40 +0900 > YAMADA Hideki wrote: > >> +import logging >> +import struct >> +from ryu.lib.packet import packet_base >> + >> + >> +LOG = logging.getLogger(__name__) > >

[Ryu-devel] [PATCH RFC] packet lib: add LLDP support.

2013-02-04 Thread YAMADA Hideki
See "ryu/tests/unit/packet/test_lldp.py" to use this library. This patch is based on Yamahata's topology discovery patch series. http://thread.gmane.org/gmane.network.ryu.devel/467 Signed-off-by: YAMADA Hideki --- ryu/lib/packet/lldp.py

  1   2   >