Re: [Ryu-devel] [PATCH v4 00/38] GRE tunnel support revised

2012-10-11 Thread Isaku Yamahata
On Fri, Oct 12, 2012 at 03:07:57PM +0900, Isaku Yamahata wrote: > Okay, I'll hard-code all of them. And let's see the result. I uploaded at git://github.com/yamahata/ryu.git contrib-ovs-import-1 ovs lib only git://github.com/yamahata/ryu.git ryu-gre-tunnel-oct-12-2012-1 ovs lib + GRE tunnel -- y

Re: [Ryu-devel] [PATCH v4 00/38] GRE tunnel support revised

2012-10-11 Thread Isaku Yamahata
On Fri, Oct 12, 2012 at 02:58:34PM +0900, FUJITA Tomonori wrote: > On Fri, 12 Oct 2012 14:55:00 +0900 > Isaku Yamahata wrote: > > > On Fri, Oct 12, 2012 at 02:32:42PM +0900, FUJITA Tomonori wrote: > >> On Fri, 12 Oct 2012 14:27:41 +0900 > >> Isaku Yamahata wrote: > >> > >> > On Fri, Oct 12, 201

Re: [Ryu-devel] [PATCH v4 00/38] GRE tunnel support revised

2012-10-11 Thread FUJITA Tomonori
On Fri, 12 Oct 2012 14:55:00 +0900 Isaku Yamahata wrote: > On Fri, Oct 12, 2012 at 02:32:42PM +0900, FUJITA Tomonori wrote: >> On Fri, 12 Oct 2012 14:27:41 +0900 >> Isaku Yamahata wrote: >> >> > On Fri, Oct 12, 2012 at 02:15:16PM +0900, Isaku Yamahata wrote: >> >> On Fri, Oct 12, 2012 at 01:21:

Re: [Ryu-devel] [PATCH v4 00/38] GRE tunnel support revised

2012-10-11 Thread Isaku Yamahata
On Fri, Oct 12, 2012 at 02:32:42PM +0900, FUJITA Tomonori wrote: > On Fri, 12 Oct 2012 14:27:41 +0900 > Isaku Yamahata wrote: > > > On Fri, Oct 12, 2012 at 02:15:16PM +0900, Isaku Yamahata wrote: > >> On Fri, Oct 12, 2012 at 01:21:39PM +0900, FUJITA Tomonori wrote: > >> > On Fri, 12 Oct 2012 12:4

Re: [Ryu-devel] [PATCH v4 00/38] GRE tunnel support revised

2012-10-11 Thread FUJITA Tomonori
On Fri, 12 Oct 2012 14:27:41 +0900 Isaku Yamahata wrote: > On Fri, Oct 12, 2012 at 02:15:16PM +0900, Isaku Yamahata wrote: >> On Fri, Oct 12, 2012 at 01:21:39PM +0900, FUJITA Tomonori wrote: >> > On Fri, 12 Oct 2012 12:42:29 +0900 >> > Isaku Yamahata wrote: >> > >> > > On Fri, Oct 12, 2012 at 0

Re: [Ryu-devel] [PATCH v4 00/38] GRE tunnel support revised

2012-10-11 Thread Isaku Yamahata
On Fri, Oct 12, 2012 at 02:15:16PM +0900, Isaku Yamahata wrote: > On Fri, Oct 12, 2012 at 01:21:39PM +0900, FUJITA Tomonori wrote: > > On Fri, 12 Oct 2012 12:42:29 +0900 > > Isaku Yamahata wrote: > > > > > On Fri, Oct 12, 2012 at 06:41:32AM +0900, FUJITA Tomonori wrote: > > >> On Thu, 11 Oct 2012

Re: [Ryu-devel] [PATCH v4 00/38] GRE tunnel support revised

2012-10-11 Thread FUJITA Tomonori
On Fri, 12 Oct 2012 14:15:16 +0900 Isaku Yamahata wrote: > On Fri, Oct 12, 2012 at 01:21:39PM +0900, FUJITA Tomonori wrote: >> On Fri, 12 Oct 2012 12:42:29 +0900 >> Isaku Yamahata wrote: >> >> > On Fri, Oct 12, 2012 at 06:41:32AM +0900, FUJITA Tomonori wrote: >> >> On Thu, 11 Oct 2012 17:17:14

Re: [Ryu-devel] [PATCH v4 00/38] GRE tunnel support revised

2012-10-11 Thread Isaku Yamahata
On Fri, Oct 12, 2012 at 01:21:39PM +0900, FUJITA Tomonori wrote: > On Fri, 12 Oct 2012 12:42:29 +0900 > Isaku Yamahata wrote: > > > On Fri, Oct 12, 2012 at 06:41:32AM +0900, FUJITA Tomonori wrote: > >> On Thu, 11 Oct 2012 17:17:14 +0900 > >> Isaku Yamahata wrote: > >> > >> > quantum and openvsw

Re: [Ryu-devel] [PATCH v4 00/38] GRE tunnel support revised

2012-10-11 Thread FUJITA Tomonori
On Fri, 12 Oct 2012 12:42:29 +0900 Isaku Yamahata wrote: > On Fri, Oct 12, 2012 at 06:41:32AM +0900, FUJITA Tomonori wrote: >> On Thu, 11 Oct 2012 17:17:14 +0900 >> Isaku Yamahata wrote: >> >> > quantum and openvswitch need patch. the patched trees are available >> > for convenience. patches de

Re: [Ryu-devel] [PATCH v4 00/38] GRE tunnel support revised

2012-10-11 Thread Isaku Yamahata
On Fri, Oct 12, 2012 at 06:41:32AM +0900, FUJITA Tomonori wrote: > On Thu, 11 Oct 2012 17:17:14 +0900 > Isaku Yamahata wrote: > > > quantum and openvswitch need patch. the patched trees are available > > for convenience. patches devstack is also avaiable. > > git://github.com/yamahata/ryu.git ryu

Re: [Ryu-devel] simple_switch in Openflow 1.2

2012-10-11 Thread FUJITA Tomonori
On Thu, 11 Oct 2012 21:54:28 -0300 Ramon Marques wrote: > I am trying to rewrite the simple_switch.py using openflow 1.2, but i can't > get the packet in_port. > In the parser of the OFPPacketIn class inside ofproto_v1_2_parser.py there > is no attribute in_port. > Where can I get the in_port whe

Re: [Ryu-devel] simple_switch in Openflow 1.2

2012-10-11 Thread Isaku Yamahata
You need to look into ofp_match for OFPXMT_OFB_IN_PORTS. Please refer to OF 1.2 spec for details. On Thu, Oct 11, 2012 at 09:54:28PM -0300, Ramon Marques wrote: > Hi, > > I am trying to rewrite the simple_switch.py using openflow 1.2, but i can't > get > the packet in_port. > In the parser of th

[Ryu-devel] simple_switch in Openflow 1.2

2012-10-11 Thread Ramon Marques
Hi, I am trying to rewrite the simple_switch.py using openflow 1.2, but i can't get the packet in_port. In the parser of the OFPPacketIn class inside ofproto_v1_2_parser.py there is no attribute in_port. Where can I get the in_port when using ofproto 1.2? thank you -- Ramon Marques Ramos En

Re: [Ryu-devel] [PATCH v4 03/38] wsgi/routes: add regex pattern for each REST component

2012-10-11 Thread FUJITA Tomonori
On Thu, 11 Oct 2012 17:17:17 +0900 Isaku Yamahata wrote: > Signed-off-by: Isaku Yamahata > --- > ryu/{lib/dpid.py => app/wsgi_path.py} | 16 ++-- > ryu/lib/dpid.py |4 +++- > ryu/lib/mac.py|2 ++ > 3 files changed, 7 insertions

Re: [Ryu-devel] [PATCH v4 05/38] ryu/exception: introduce more exception for later use

2012-10-11 Thread FUJITA Tomonori
On Thu, 11 Oct 2012 17:17:19 +0900 Isaku Yamahata wrote: > MacAddressAlreadyExists and MacAddressNotFound > > Signed-off-by: Isaku Yamahata > --- > ryu/exception.py | 17 + > 1 file changed, 17 insertions(+) > > diff --git a/ryu/exception.py b/ryu/exception.py > index 0a1e72

Re: [Ryu-devel] [PATCH v4 33/38] bin/ryu-manager: work around gflags of quantum_adapter

2012-10-11 Thread FUJITA Tomonori
On Thu, 11 Oct 2012 17:17:47 +0900 Isaku Yamahata wrote: > Temporal work around. This will be addressed later. How can be solved in the future? -- Don't let slow site performance ruin your business. Deploy New Relic APM

Re: [Ryu-devel] [PATCH v4 10/38] dpset: add port{add, delete, modify} event for convenience and helper functions

2012-10-11 Thread FUJITA Tomonori
On Thu, 11 Oct 2012 17:17:24 +0900 Isaku Yamahata wrote: > For discovery, it's interested only in datapath/port appearance/disappearance. > With this, discovery app would not have to handle OFP events directly. Can you describe how this is necessary for the GRE app, not for discovery app?

Re: [Ryu-devel] [PATCH v4 16/38] app/rest_tunnel: REST API for tunnel

2012-10-11 Thread FUJITA Tomonori
On Thu, 11 Oct 2012 17:17:30 +0900 Isaku Yamahata wrote: > Signed-off-by: Isaku Yamahata > --- > Changes > v2 -> v3: > - de-GRE > - add requirements of routes > --- > ryu/app/rest_tunnel.py | 210 > > 1 file changed, 210 insertions(+) > create

Re: [Ryu-devel] [PATCH v4 01/38] utils: avoid double import and don't append sys.path with same path

2012-10-11 Thread FUJITA Tomonori
On Thu, 11 Oct 2012 17:17:15 +0900 Isaku Yamahata wrote: > Since import_module() isn't aware of cwd, so it may result in double import. > Avoid double import. Elaborate how it happens, please. > sys.path can be populated with same path. Don't append when path is already > added. > > Signed-off

Re: [Ryu-devel] [PATCH v4 20/38] controller/switch_conf: add interface to store per-switch configuration

2012-10-11 Thread FUJITA Tomonori
On Thu, 11 Oct 2012 17:17:34 +0900 Isaku Yamahata wrote: > Signed-off-by: Isaku Yamahata > > --- > Changes v1 -> v2: > - generate events when values are set/changed/deleted > - added __str__ method for debug > --- > ryu/controller/conf_switch.py | 105 > ++

Re: [Ryu-devel] [PATCH v4 28/38] lib/ovs/vsctl: python reimplementation of ovs-vsctl

2012-10-11 Thread FUJITA Tomonori
On Thu, 11 Oct 2012 17:17:42 +0900 Isaku Yamahata wrote: > Signed-off-by: Isaku Yamahata > --- > ryu/lib/ovs/vsctl.py | 1601 > ++ > 1 file changed, 1601 insertions(+) > create mode 100644 ryu/lib/ovs/vsctl.py Can we have _only_ what are necess

Re: [Ryu-devel] [PATCH v4 34/38] ryu/app/ovsdb_client: easy test app for lib/ovs/db_client.py

2012-10-11 Thread FUJITA Tomonori
On Thu, 11 Oct 2012 17:17:48 +0900 Isaku Yamahata wrote: > Signed-off-by: Isaku Yamahata > --- > ryu/app/ovsdb_client.py | 44 > 1 file changed, 44 insertions(+) > create mode 100644 ryu/app/ovsdb_client.py Don't put test application to ryu/app.

Re: [Ryu-devel] [PATCH v4 35/38] ryu/app/ovs_vsctl: test program for lib/ovs/vsctl.py

2012-10-11 Thread FUJITA Tomonori
On Thu, 11 Oct 2012 17:17:49 +0900 Isaku Yamahata wrote: > Signed-off-by: Isaku Yamahata > --- > ryu/app/ovs_vsctl.py | 47 +++ > 1 file changed, 47 insertions(+) > create mode 100644 ryu/app/ovs_vsctl.py As I wrote in another mail, drop it for no

Re: [Ryu-devel] [PATCH v4 00/38] GRE tunnel support revised

2012-10-11 Thread FUJITA Tomonori
On Thu, 11 Oct 2012 17:17:14 +0900 Isaku Yamahata wrote: > quantum and openvswitch need patch. the patched trees are available > for convenience. patches devstack is also avaiable. > git://github.com/yamahata/ryu.git ryu-gre-tunnel-oct-11-2012 > git://github.com/yamahata/openvswitch.git ryu-gre-t

Re: [Ryu-devel] [PATCH] packet lib: fix ipv4 assertion of option field

2012-10-11 Thread FUJITA Tomonori
On Thu, 11 Oct 2012 16:25:03 +0900 HIYAMA Manabu wrote: > > Signed-off-by: HIYAMA Manabu > --- > ryu/lib/packet/ipv4.py |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Applied. -- Don't let slow site perfo

Re: [Ryu-devel] [PATCH] packet lib: fix udp header length

2012-10-11 Thread FUJITA Tomonori
On Thu, 11 Oct 2012 17:19:57 +0900 HIYAMA Manabu wrote: > Fix the f5d2157 commit. > > Signed-off-by: HIYAMA Manabu > --- > ryu/lib/packet/udp.py | 22 -- > 1 file changed, 12 insertions(+), 10 deletions(-) Applied. ---

Re: [Ryu-devel] [PATCH] test: add unittests for packet library

2012-10-11 Thread FUJITA Tomonori
On Thu, 11 Oct 2012 17:54:37 +0900 HIYAMA Manabu wrote: > > Signed-off-by: HIYAMA Manabu > --- > ryu/tests/unit/packet/test_arp.py | 176 > > ryu/tests/unit/packet/test_vlan.py | 139 > tools/test-requires|

Re: [Ryu-devel] [PATCH] tools/pip-requires: add routes which is used by wsgi.py module

2012-10-11 Thread FUJITA Tomonori
On Thu, 11 Oct 2012 12:57:05 +0900 Isaku Yamahata wrote: > Signed-off-by: Isaku Yamahata > --- > tools/pip-requires |1 + > 1 file changed, 1 insertion(+) Applied. -- Don't let slow site performance ruin your busi

[Ryu-devel] [PATCH] test: add unittests for packet library

2012-10-11 Thread HIYAMA Manabu
Signed-off-by: HIYAMA Manabu --- ryu/tests/unit/packet/test_arp.py | 176 ryu/tests/unit/packet/test_vlan.py | 139 tools/test-requires|1 + 3 files changed, 316 insertions(+) create mode 100644 ryu/tests/u

Re: [Ryu-devel] [PATCH v4 00/38] GRE tunnel support revised

2012-10-11 Thread Isaku Yamahata
Oops, I didn't intended to post patches of 34-37. Please skip 34-37 when reviewing. On Thu, Oct 11, 2012 at 05:17:14PM +0900, Isaku Yamahata wrote: > This is a revised version of GRE tunnel support, which is greatly simplified > from the previous version. > > Changes v3 -> v4: > - minor bug fixe

[Ryu-devel] [PATCH] packet lib: fix udp header length

2012-10-11 Thread HIYAMA Manabu
Fix the f5d2157 commit. Signed-off-by: HIYAMA Manabu --- ryu/lib/packet/udp.py | 22 -- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/ryu/lib/packet/udp.py b/ryu/lib/packet/udp.py index b949b5c..c4fe529 100644 --- a/ryu/lib/packet/udp.py +++ b/ryu/lib/pack

[Ryu-devel] [PATCH v4 31/38] app/tunnel_port_updater: application that creates/deletes tunnel ports

2012-10-11 Thread Isaku Yamahata
This application dynamically creates/deletes tunnel ports for GRE tunnel app. Signed-off-by: Isaku Yamahata --- Changes v3 -> v4: - 'key=flow' option to tunnel port --- ryu/app/tunnel_port_updater.py | 470 1 file changed, 470 insertions(+) create mode

[Ryu-devel] [PATCH v4 19/38] app/gre_tunnel: implement GRETunnel app

2012-10-11 Thread Isaku Yamahata
- race masking layer - debug app: PortSetDebug - implement GRETunnel app: app/gre_tunnel Signed-off-by: Isaku Yamahata --- Changes v3 -> v4: - duplicated output to tunnel port --- ryu/app/gre_tunnel.py | 928 + 1 file changed, 928 insertions(+) c

[Ryu-devel] [PATCH v4 25/38] ryu/controller/network.py: add helper functions for tunnel updater

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- ryu/controller/network.py | 17 + 1 file changed, 13 insertions(+), 4 deletions(-) diff --git a/ryu/controller/network.py b/ryu/controller/network.py index 911b1fe..c428221 100644 --- a/ryu/controller/network.py +++ b/ryu/controller/network.py

[Ryu-devel] [PATCH v4 34/38] ryu/app/ovsdb_client: easy test app for lib/ovs/db_client.py

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- ryu/app/ovsdb_client.py | 44 1 file changed, 44 insertions(+) create mode 100644 ryu/app/ovsdb_client.py diff --git a/ryu/app/ovsdb_client.py b/ryu/app/ovsdb_client.py new file mode 100644 index 000..6a95539 -

[Ryu-devel] [PATCH v4 16/38] app/rest_tunnel: REST API for tunnel

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- Changes v2 -> v3: - de-GRE - add requirements of routes --- ryu/app/rest_tunnel.py | 210 1 file changed, 210 insertions(+) create mode 100644 ryu/app/rest_tunnel.py diff --git a/ryu/app/rest_tunnel.py b/ryu/app

[Ryu-devel] [PATCH v4 38/38] docs: tunneling app

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- Changes v3 -> v4: - clarification - some more devstack variables --- doc/source/openstack.rst|1 + doc/source/tunneling_with_openstack.rst | 324 +++ 2 files changed, 325 insertions(+) create mode 100644 doc/sourc

[Ryu-devel] [PATCH v4 33/38] bin/ryu-manager: work around gflags of quantum_adapter

2012-10-11 Thread Isaku Yamahata
Temporal work around. This will be addressed later. Signed-off-by: Isaku Yamahata --- bin/ryu-manager |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/bin/ryu-manager b/bin/ryu-manager index 4d6c490..94663aa 100755 --- a/bin/ryu-manager +++ b/bin/ryu-manager @@ -33,6 +33

[Ryu-devel] [PATCH v4 17/38] app/client: add tunnel client

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- Changes v2 -> v3: - de-GRE --- ryu/app/client.py | 46 ++ 1 file changed, 46 insertions(+) diff --git a/ryu/app/client.py b/ryu/app/client.py index b8e5096..de74357 100644 --- a/ryu/app/client.py +++ b/ryu/app/client

[Ryu-devel] [PATCH v4 26/38] lib/ovs/db_client: library for low-level ovsdb manipulation

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- ryu/lib/ovs/db_client.py | 132 ++ 1 file changed, 132 insertions(+) create mode 100644 ryu/lib/ovs/db_client.py diff --git a/ryu/lib/ovs/db_client.py b/ryu/lib/ovs/db_client.py new file mode 100644 index 000..ec

[Ryu-devel] [PATCH v4 37/38] etc/ryu/ryu.conf: local modification

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- etc/ryu/ryu.conf |5 + 1 file changed, 5 insertions(+) diff --git a/etc/ryu/ryu.conf b/etc/ryu/ryu.conf index 3b6ca1b..93fbea4 100644 --- a/etc/ryu/ryu.conf +++ b/etc/ryu/ryu.conf @@ -4,3 +4,8 @@ #--wsapi_port= #--ofp_listen_host= #--ofp_listen_port=

[Ryu-devel] [PATCH v4 35/38] ryu/app/ovs_vsctl: test program for lib/ovs/vsctl.py

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- ryu/app/ovs_vsctl.py | 47 +++ 1 file changed, 47 insertions(+) create mode 100644 ryu/app/ovs_vsctl.py diff --git a/ryu/app/ovs_vsctl.py b/ryu/app/ovs_vsctl.py new file mode 100644 index 000..75c651c --- /dev/n

[Ryu-devel] [PATCH v4 15/38] controller/tunnel: introduce new class that tracks infos related to tunneling

2012-10-11 Thread Isaku Yamahata
- helper functions and event generator for gre tunnel - plug events for gre tunnel app Signed-off-by: Isaku Yamahata --- ryu/controller/tunnels.py | 197 + 1 file changed, 197 insertions(+) create mode 100644 ryu/controller/tunnels.py diff --git a/r

[Ryu-devel] [PATCH v4 20/38] controller/switch_conf: add interface to store per-switch configuration

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- Changes v1 -> v2: - generate events when values are set/changed/deleted - added __str__ method for debug --- ryu/controller/conf_switch.py | 105 + 1 file changed, 105 insertions(+) create mode 100644 ryu/controller/conf

[Ryu-devel] [PATCH v4 36/38] local: local modification to test ovsdb_client, ovs_vsctl

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- bin/ryu-manager |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/bin/ryu-manager b/bin/ryu-manager index 94663aa..8cfea43 100755 --- a/bin/ryu-manager +++ b/bin/ryu-manager @@ -57,7 +57,10 @@ def main(): # always enable ofp for now.

[Ryu-devel] [PATCH v4 32/38] app: Add quantum adapter

2012-10-11 Thread Isaku Yamahata
From: Yoshihiro Kaneko Signed-off-by: Yoshihiro Kaneko Signed-off-by: Isaku Yamahata --- ryu/app/quantum_adapter.py | 580 1 file changed, 580 insertions(+) create mode 100644 ryu/app/quantum_adapter.py diff --git a/ryu/app/quantum_adapter.py b/r

[Ryu-devel] [PATCH v4 14/38] bin/ryu-client: support mac address tracking

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- bin/ryu-client |6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/bin/ryu-client b/bin/ryu-client index dc1f697..20eeff6 100755 --- a/bin/ryu-client +++ b/bin/ryu-client @@ -42,7 +42,11 @@ def client_test(): 'list_ports': lambda

[Ryu-devel] [PATCH v4 21/38] ryu/app: REST API to set per-switch configuration

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- ryu/app/conf_switch_key.py | 18 + ryu/app/rest_conf_switch.py | 173 +++ 2 files changed, 191 insertions(+) create mode 100644 ryu/app/conf_switch_key.py create mode 100644 ryu/app/rest_conf_switch.py diff --git

[Ryu-devel] [PATCH v4 22/38] ryu/app/client: support conf_switch

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata Conflicts: ryu/app/client.py --- ryu/app/client.py | 44 ++-- 1 file changed, 42 insertions(+), 2 deletions(-) diff --git a/ryu/app/client.py b/ryu/app/client.py index de74357..04d5fec 100644 --- a/ryu/app/client.p

[Ryu-devel] [PATCH v4 07/38] ryu/app/client: factor out rest client code

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- ryu/app/client.py | 53 +++-- 1 file changed, 31 insertions(+), 22 deletions(-) diff --git a/ryu/app/client.py b/ryu/app/client.py index c472889..d54fe44 100644 --- a/ryu/app/client.py +++ b/ryu/app/client.py @@

[Ryu-devel] [PATCH v4 23/38] bin/ryu-client: support switch_conf

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata Conflicts: bin/ryu-client --- bin/ryu-client | 10 ++ 1 file changed, 10 insertions(+) diff --git a/bin/ryu-client b/bin/ryu-client index e121f61..04fec26 100755 --- a/bin/ryu-client +++ b/bin/ryu-client @@ -21,6 +21,7 @@ from optparse import Opt

[Ryu-devel] [PATCH v4 29/38] ovs/bridge: helper class to manipulate ovs bridge

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- ryu/lib/ovs/bridge.py | 228 + 1 file changed, 228 insertions(+) create mode 100644 ryu/lib/ovs/bridge.py diff --git a/ryu/lib/ovs/bridge.py b/ryu/lib/ovs/bridge.py new file mode 100644 index 000..df157be ---

[Ryu-devel] [PATCH v4 30/38] lib/synchronized: introduce helper decorator to lock

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- ryu/lib/synchronized.py | 29 + 1 file changed, 29 insertions(+) create mode 100644 ryu/lib/synchronized.py diff --git a/ryu/lib/synchronized.py b/ryu/lib/synchronized.py new file mode 100644 index 000..d1129db --- /dev/null ++

[Ryu-devel] [PATCH v4 10/38] dpset: add port{add, delete, modify} event for convenience and helper functions

2012-10-11 Thread Isaku Yamahata
For discovery, it's interested only in datapath/port appearance/disappearance. With this, discovery app would not have to handle OFP events directly. Signed-off-by: Isaku Yamahata --- Changes v3 -> v4: - don't use cork/uncork stuff. Changes v2 -> v3: - track port status change properly So far

[Ryu-devel] [PATCH v4 18/38] bin/ryu-client: support gre tunnel client

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- Changes v2 -> v3: - de-GRE --- bin/ryu-client | 21 - 1 file changed, 20 insertions(+), 1 deletion(-) diff --git a/bin/ryu-client b/bin/ryu-client index 20eeff6..e121f61 100755 --- a/bin/ryu-client +++ b/bin/ryu-client @@ -19,6 +19,7 @@ im

[Ryu-devel] [PATCH v4 09/38] controller/network: factor out network.py and add event generator for gre tunnel and helper methods

2012-10-11 Thread Isaku Yamahata
- Factor out from network.py the logic to track network and dpid - introduce class Port to allow other info Later we'll track mac address associated to port. So allow intermediate layer to contain infos related to Port. - make network track mac address associated to port - generate events Sign

[Ryu-devel] [PATCH v4 27/38] lib/ovs: add constants for ovsdb Open_vSwitch db

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- ryu/lib/ovs/vswitch_idl.py | 172 1 file changed, 172 insertions(+) create mode 100644 ryu/lib/ovs/__init__.py create mode 100644 ryu/lib/ovs/vswitch_idl.py diff --git a/ryu/lib/ovs/__init__.py b/ryu/lib/ovs/__init

[Ryu-devel] [PATCH v4 13/38] app/client: add mac support

2012-10-11 Thread Isaku Yamahata
- fix path to 'networks' - list_macs() must return string object, not implicit None Signed-off-by: Isaku Yamahata --- ryu/app/client.py | 31 +++ 1 file changed, 23 insertions(+), 8 deletions(-) diff --git a/ryu/app/client.py b/ryu/app/client.py index fcf1283..b8e5

[Ryu-devel] [PATCH v4 06/38] app/rest_nw_id: add port type for reserved port and vport-gre

2012-10-11 Thread Isaku Yamahata
This patch adds new port type for reserved port and vport-gre. They will be used later. Signed-off-by: Isaku Yamahata --- ryu/app/rest_nw_id.py | 20 +++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/ryu/app/rest_nw_id.py b/ryu/app/rest_nw_id.py index cb69103..

[Ryu-devel] [PATCH v4 11/38] app/rest: add API to register/update mac address for a given port

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- ryu/app/rest.py | 72 +++ 1 file changed, 67 insertions(+), 5 deletions(-) diff --git a/ryu/app/rest.py b/ryu/app/rest.py index c9e649c..3fd045e 100644 --- a/ryu/app/rest.py +++ b/ryu/app/rest.py @@ -21,9 +21

[Ryu-devel] [PATCH v4 12/38] app/rest: add requirements to path component

2012-10-11 Thread Isaku Yamahata
This patch adds requirements to path component. And simplify with submapper. Signed-off-by: Isaku Yamahata --- ryu/app/rest.py | 36 ++-- 1 file changed, 22 insertions(+), 14 deletions(-) diff --git a/ryu/app/rest.py b/ryu/app/rest.py index 3fd045e..52c147a 100

[Ryu-devel] [PATCH v4 08/38] app/client: add helper function to ignore NOT_FOUND, 404

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- ryu/app/client.py | 13 + 1 file changed, 13 insertions(+) diff --git a/ryu/app/client.py b/ryu/app/client.py index d54fe44..fcf1283 100644 --- a/ryu/app/client.py +++ b/ryu/app/client.py @@ -18,6 +18,19 @@ import httplib import urlparse +def

[Ryu-devel] [PATCH v4 24/38] exception: Add OVSBridgeNotFound exception

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- ryu/exception.py |3 +++ 1 file changed, 3 insertions(+) diff --git a/ryu/exception.py b/ryu/exception.py index aa9597e..45fbd87 100644 --- a/ryu/exception.py +++ b/ryu/exception.py @@ -79,3 +79,6 @@ class TunnelKeyAlreadyExist(RyuException): class Tunnel

[Ryu-devel] [PATCH v4 05/38] ryu/exception: introduce more exception for later use

2012-10-11 Thread Isaku Yamahata
MacAddressAlreadyExists and MacAddressNotFound Signed-off-by: Isaku Yamahata --- ryu/exception.py | 17 + 1 file changed, 17 insertions(+) diff --git a/ryu/exception.py b/ryu/exception.py index 0a1e72c..aa9597e 100644 --- a/ryu/exception.py +++ b/ryu/exception.py @@ -60,5 +60,

[Ryu-devel] [PATCH v4 00/38] GRE tunnel support revised

2012-10-11 Thread Isaku Yamahata
This is a revised version of GRE tunnel support, which is greatly simplified from the previous version. Changes v3 -> v4: - minor bug fixes - include synchronization.py - documentation update - instance metadata Changes v2 -> v3: - pylint - fixed some race condition Changes v1 -> v2: - eliminat

[Ryu-devel] [PATCH v4 02/38] ryu/controller/controller: add a helper method, is_reserved_port()

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- ryu/controller/controller.py |3 +++ 1 file changed, 3 insertions(+) diff --git a/ryu/controller/controller.py b/ryu/controller/controller.py index dd5f16b..bc118c8 100644 --- a/ryu/controller/controller.py +++ b/ryu/controller/controller.py @@ -261,6 +261,9

[Ryu-devel] [PATCH v4 01/38] utils: avoid double import and don't append sys.path with same path

2012-10-11 Thread Isaku Yamahata
Since import_module() isn't aware of cwd, so it may result in double import. Avoid double import. sys.path can be populated with same path. Don't append when path is already added. Signed-off-by: Isaku Yamahata --- ryu/utils.py | 27 +-- 1 file changed, 21 insertions(+)

[Ryu-devel] [PATCH v4 04/38] app/rest: add requirements to path component

2012-10-11 Thread Isaku Yamahata
This patch adds requirements to path component. And simplify with submapper. Signed-off-by: Isaku Yamahata --- ryu/app/rest.py | 66 ++- 1 file changed, 36 insertions(+), 30 deletions(-) diff --git a/ryu/app/rest.py b/ryu/app/rest.py index c

[Ryu-devel] [PATCH v4 03/38] wsgi/routes: add regex pattern for each REST component

2012-10-11 Thread Isaku Yamahata
Signed-off-by: Isaku Yamahata --- ryu/{lib/dpid.py => app/wsgi_path.py} | 16 ++-- ryu/lib/dpid.py |4 +++- ryu/lib/mac.py|2 ++ 3 files changed, 7 insertions(+), 15 deletions(-) copy ryu/{lib/dpid.py => app/wsgi_path.py} (69%)

[Ryu-devel] [PATCH] packet lib: fix ipv4 assertion of option field

2012-10-11 Thread HIYAMA Manabu
Signed-off-by: HIYAMA Manabu --- ryu/lib/packet/ipv4.py |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ryu/lib/packet/ipv4.py b/ryu/lib/packet/ipv4.py index d3cc4b3..99140d4 100644 --- a/ryu/lib/packet/ipv4.py +++ b/ryu/lib/packet/ipv4.py @@ -73,7 +73,7 @@ class ipv4(pack