[Ryu-devel] [PATCH 0/2] avoid picking up older installation of ryu

2013-08-22 Thread YAMAMOTO Takashi
this is after move-to-entrypoints patches i posted while ago. YAMAMOTO Takashi (2): new way to use pbr avoid pbr's non multi-version aware script ryu/hooks.py | 11 +++ setup.cfg| 1 - setup.py | 4 ++-- 3 files changed, 13 insertions(+), 3 deletions(-) -- 1.8.3.1 -

[Ryu-devel] [PATCH 1/2] new way to use pbr

2013-08-22 Thread YAMAMOTO Takashi
Signed-off-by: YAMAMOTO Takashi --- setup.cfg | 1 - setup.py | 4 ++-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/setup.cfg b/setup.cfg index fd49774..a3d3329 100644 --- a/setup.cfg +++ b/setup.cfg @@ -43,7 +43,6 @@ doc_files = LICENSE [global] setup-hooks = -pbr.hoo

[Ryu-devel] [PATCH 2/2] avoid pbr's non multi-version aware script

2013-08-22 Thread YAMAMOTO Takashi
otherwise the latest ryu-manager can pick up older modules if multiple versions of ryu is installed on a system. Signed-off-by: YAMAMOTO Takashi --- ryu/hooks.py | 11 +++ 1 file changed, 11 insertions(+) diff --git a/ryu/hooks.py b/ryu/hooks.py index c2414cd..96e23aa 100644 --- a/ryu/h

Re: [Ryu-devel] [PATCH 2/2] Actions is a list

2013-08-22 Thread Satoshi Kobayashi
Hi Ohmura-san, Thank you for your review. ofctl_* is very useful and my favorite. Thanks! Satoshi 2013/8/20 Kei Ohmura > Sorry, it's my mistake. The below patch looks good to me. > > 2013/8/8 Satoshi Kobayashi : > > > > Signed-off-by: Satoshi Kobayashi > > --- > > ryu/lib/ofctl_v1_0.py |

[Ryu-devel] [PATCH 17/19] test_parser_v12: update after OFPTableStats.name change

2013-08-22 Thread YAMAMOTO Takashi
Signed-off-by: YAMAMOTO Takashi --- ryu/tests/unit/ofproto/test_parser_v12.py | 1 - 1 file changed, 1 deletion(-) diff --git a/ryu/tests/unit/ofproto/test_parser_v12.py b/ryu/tests/unit/ofproto/test_parser_v12.py index a783365..cf09d26 100644 --- a/ryu/tests/unit/ofproto/test_parser_v12.py +++

[Ryu-devel] [PATCH 12/19] of13: remove trailing NULs in OFPPort.name

2013-08-22 Thread YAMAMOTO Takashi
Signed-off-by: YAMAMOTO Takashi --- ryu/ofproto/ofproto_v1_3_parser.py | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/ryu/ofproto/ofproto_v1_3_parser.py b/ryu/ofproto/ofproto_v1_3_parser.py index dc1b9de..a0475df 100644 --- a/ryu/ofproto/ofproto_v1_3_parser.py ++

[Ryu-devel] [PATCH 09/19] of10: update expected *.json output after OFPPhyPort.name change

2013-08-22 Thread YAMAMOTO Takashi
this also includes updates after "packet_data_generator: fix port names" commit. Signed-off-by: YAMAMOTO Takashi --- ryu/tests/unit/ofproto/json/of10/1-6-ofp_switch_features.packet.json | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ryu/tests/unit/ofproto/json/of10/1-6-

[Ryu-devel] [PATCH 10/19] of12: remove trailing NULs in OFPPort.name

2013-08-22 Thread YAMAMOTO Takashi
Signed-off-by: YAMAMOTO Takashi --- ryu/ofproto/ofproto_v1_2_parser.py | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/ryu/ofproto/ofproto_v1_2_parser.py b/ryu/ofproto/ofproto_v1_2_parser.py index b1957d3..0d6f245 100644 --- a/ryu/ofproto/ofproto_v1_2_parser.py ++

[Ryu-devel] [PATCH 13/19] of13: update expected *.json output after OFPPort.name change

2013-08-22 Thread YAMAMOTO Takashi
this also includes updates after "packet_data_generator: fix port names" commit. Signed-off-by: YAMAMOTO Takashi --- ryu/tests/unit/ofproto/json/of13/4-39-ofp_port_status.packet.json | 2 +- ryu/tests/unit/ofproto/json/of13/4-54-ofp_port_desc_reply.packet.json | 4 ++-- 2 files changed, 3 in

[Ryu-devel] [PATCH 08/19] of10: remove trailing NULs in OFPPhyPort.name

2013-08-22 Thread YAMAMOTO Takashi
Signed-off-by: YAMAMOTO Takashi --- ryu/ofproto/ofproto_v1_0_parser.py | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/ryu/ofproto/ofproto_v1_0_parser.py b/ryu/ofproto/ofproto_v1_0_parser.py index 84fba98..e6c94b2 100644 --- a/ryu/ofproto/ofproto_v1_0_parser.py ++

[Ryu-devel] [PATCH 03/19] of12: update expected *.json after OFPDescStats changes

2013-08-22 Thread YAMAMOTO Takashi
Signed-off-by: YAMAMOTO Takashi --- .../ofproto/json/of12/3-0-ofp_desc_stats_reply.packet.json | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/ryu/tests/unit/ofproto/json/of12/3-0-ofp_desc_stats_reply.packet.json b/ryu/tests/unit/ofproto/json/of12/3-0-ofp_desc

[Ryu-devel] [PATCH 19/19] of13: update expected *.json output for OFPTableFeaturesStats.name

2013-08-22 Thread YAMAMOTO Takashi
Signed-off-by: YAMAMOTO Takashi --- .../of13/4-56-ofp_table_features_reply.packet.json | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/ryu/tests/unit/ofproto/json/of13/4-56-ofp_table_features_reply.packet.json b/ryu/tests/unit/ofproto/json/of13/4-56-

[Ryu-devel] [PATCH 15/19] of12: remove trailing NULs in OFPTableStats.name

2013-08-22 Thread YAMAMOTO Takashi
Signed-off-by: YAMAMOTO Takashi --- ryu/ofproto/ofproto_v1_2_parser.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/ryu/ofproto/ofproto_v1_2_parser.py b/ryu/ofproto/ofproto_v1_2_parser.py index 0d6f245..d695244 100644 --- a/ryu/ofproto/ofproto_v1_2_parser.py +++ b/ryu/ofproto/ofproto_v

[Ryu-devel] [PATCH 11/19] of12: update expected *.json output after OFPPort.name change

2013-08-22 Thread YAMAMOTO Takashi
this also includes updates after "packet_data_generator: fix port names" commit. Signed-off-by: YAMAMOTO Takashi --- ryu/tests/unit/ofproto/json/of12/3-39-ofp_port_status.packet.json | 2 +- ryu/tests/unit/ofproto/json/of12/3-6-ofp_features_reply.packet.json | 4 ++-- 2 files changed, 3 insert

[Ryu-devel] [PATCH 07/19] regen after "packet_data_generator: fix port names" commit

2013-08-22 Thread YAMAMOTO Takashi
Signed-off-by: YAMAMOTO Takashi --- .../packet_data/of10/1-6-ofp_switch_features.packet | Bin 128 -> 128 bytes ryu/tests/packet_data/of12/3-39-ofp_port_status.packet | Bin 80 -> 80 bytes .../packet_data/of12/3-6-ofp_features_reply.packet | Bin 160 -> 160 bytes ryu/tests/packet_data/o

[Ryu-devel] [PATCH 02/19] of12: make OFPDescStats more human readable

2013-08-22 Thread YAMAMOTO Takashi
- when parsing, remove zero padding. - stop base64 encoding for json representation because these fields are ASCII strings per OF spec. Signed-off-by: YAMAMOTO Takashi --- ryu/ofproto/ofproto_v1_2_parser.py | 11 +++ 1 file changed, 11 insertions(+) diff --git a/ryu/ofproto/ofproto_v1

[Ryu-devel] [PATCH 06/19] packet_data_generator: fix port names

2013-08-22 Thread YAMAMOTO Takashi
these broken names were come from the bug in LINC. https://github.com/FlowForwarding/LINC-Switch/pull/133 Signed-off-by: YAMAMOTO Takashi --- ryu/tests/packet_data_generator/src/x1.erl | 4 ++-- ryu/tests/packet_data_generator/src/x3.erl | 6 +++--- ryu/tests/packet_data_generator/src/x4.erl | 2

[Ryu-devel] [PATCH 00/19] remove trailing nul from some OF fixed length strings

2013-08-22 Thread YAMAMOTO Takashi
this set is after OFPPort.hw_addr changes i posted last week. YAMAMOTO Takashi (19): of10: make OFPDescStats more human readable of12: make OFPDescStats more human readable of12: update expected *.json after OFPDescStats changes of13: make OFPDescStats more human readable of13: update ex

[Ryu-devel] [PATCH 14/19] of10: remove trailing NULs in OFPTableStats.name

2013-08-22 Thread YAMAMOTO Takashi
Signed-off-by: YAMAMOTO Takashi --- ryu/ofproto/ofproto_v1_0_parser.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/ryu/ofproto/ofproto_v1_0_parser.py b/ryu/ofproto/ofproto_v1_0_parser.py index e6c94b2..d8e42bb 100644 --- a/ryu/ofproto/ofproto_v1_0_parser.py +++ b/ryu/ofproto/ofproto_v

[Ryu-devel] [PATCH 04/19] of13: make OFPDescStats more human readable

2013-08-22 Thread YAMAMOTO Takashi
- when parsing, remove zero padding. - stop base64 encoding for json representation because these fields are ASCII strings per OF spec. Signed-off-by: YAMAMOTO Takashi --- ryu/ofproto/ofproto_v1_3_parser.py | 11 +++ 1 file changed, 11 insertions(+) diff --git a/ryu/ofproto/ofproto_v1

[Ryu-devel] [PATCH 18/19] of13: remove trailing NULs in OFPTableFeaturesStats.name

2013-08-22 Thread YAMAMOTO Takashi
Signed-off-by: YAMAMOTO Takashi --- ryu/ofproto/ofproto_v1_3_parser.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ryu/ofproto/ofproto_v1_3_parser.py b/ryu/ofproto/ofproto_v1_3_parser.py index a0475df..f6c4058 100644 --- a/ryu/ofproto/ofproto_v1_3_parser.py +++ b/ryu/of

[Ryu-devel] [PATCH 01/19] of10: make OFPDescStats more human readable

2013-08-22 Thread YAMAMOTO Takashi
- when parsing, remove zero padding. - stop base64 encoding for json representation because these fields are ASCII strings per OF spec. Signed-off-by: YAMAMOTO Takashi --- ryu/ofproto/ofproto_v1_0_parser.py | 11 +++ 1 file changed, 11 insertions(+) diff --git a/ryu/ofproto/ofproto_v1

[Ryu-devel] [PATCH 05/19] of13: update expected *.json after OFPDescStats changes

2013-08-22 Thread YAMAMOTO Takashi
Signed-off-by: YAMAMOTO Takashi --- .../unit/ofproto/json/of13/4-0-ofp_desc_reply.packet.json | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/ryu/tests/unit/ofproto/json/of13/4-0-ofp_desc_reply.packet.json b/ryu/tests/unit/ofproto/json/of13/4-0-ofp_desc_reply.

Re: [Ryu-devel] [PATCH 01/16] of10: use text address for OFPPhyPort.hw_addr and OFPPortMod.hw_addr

2013-08-22 Thread YAMAMOTO Takashi
> On Fri, 23 Aug 2013 12:18:31 +0900 (JST) > yamam...@valinux.co.jp (YAMAMOTO Takashi) wrote: > >>> On Fri, 16 Aug 2013 15:29:22 +0900 >>> YAMAMOTO Takashi wrote: >>> Signed-off-by: YAMAMOTO Takashi --- ryu/ofproto/ofproto_v1_0_parser.py | 6 +- 1 file changed, 5 insert

Re: [Ryu-devel] [PATCH 01/16] of10: use text address for OFPPhyPort.hw_addr and OFPPortMod.hw_addr

2013-08-22 Thread FUJITA Tomonori
On Fri, 23 Aug 2013 12:18:31 +0900 (JST) yamam...@valinux.co.jp (YAMAMOTO Takashi) wrote: >> On Fri, 16 Aug 2013 15:29:22 +0900 >> YAMAMOTO Takashi wrote: >> >>> Signed-off-by: YAMAMOTO Takashi >>> --- >>> ryu/ofproto/ofproto_v1_0_parser.py | 6 +- >>> 1 file changed, 5 insertions(+), 1 de

Re: [Ryu-devel] [PATCH 01/16] of10: use text address for OFPPhyPort.hw_addr and OFPPortMod.hw_addr

2013-08-22 Thread YAMAMOTO Takashi
> On Fri, 16 Aug 2013 15:29:22 +0900 > YAMAMOTO Takashi wrote: > >> Signed-off-by: YAMAMOTO Takashi >> --- >> ryu/ofproto/ofproto_v1_0_parser.py | 6 +- >> 1 file changed, 5 insertions(+), 1 deletion(-) >> >> diff --git a/ryu/ofproto/ofproto_v1_0_parser.py >> b/ryu/ofproto/ofproto_v1_0_pa

[Ryu-devel] [PATCH 2/2] add simple learning switch app for OF1.2

2013-08-22 Thread FUJITA Tomonori
Signed-off-by: FUJITA Tomonori --- ryu/app/simple_switch_12.py | 87 + 1 file changed, 87 insertions(+) create mode 100644 ryu/app/simple_switch_12.py diff --git a/ryu/app/simple_switch_12.py b/ryu/app/simple_switch_12.py new file mode 100644 index 00

[Ryu-devel] [PATCH 1/2] add simple learning switch app for OF1.3

2013-08-22 Thread FUJITA Tomonori
Signed-off-by: FUJITA Tomonori --- ryu/app/simple_switch_13.py | 87 + 1 file changed, 87 insertions(+) create mode 100644 ryu/app/simple_switch_13.py diff --git a/ryu/app/simple_switch_13.py b/ryu/app/simple_switch_13.py new file mode 100644 index 00

[Ryu-devel] [PATCH 0/2] add example simple learning switch apps for OF1.2/1.3

2013-08-22 Thread FUJITA Tomonori
Hi, This adds simple_switch applications for OF1.2/13 (mac learning switch). They are examples for new developers so we like to keep them as simple as possible. IOW, no plan to add features. The code uses the new APIs of OF match and our packet library. So it might worth checking even for develop

[Ryu-devel] [PATCH] of12: set ActionOutput default max_len to zero

2013-08-22 Thread FUJITA Tomonori
same as of10. Signed-off-by: FUJITA Tomonori --- ryu/ofproto/ofproto_v1_2_parser.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ryu/ofproto/ofproto_v1_2_parser.py b/ryu/ofproto/ofproto_v1_2_parser.py index e6dc5c6..c40533e 100644 --- a/ryu/ofproto/ofproto_v1_2_parser.py

[Ryu-devel] [PATCH] of13: set ActionOutput default max_len to zero

2013-08-22 Thread FUJITA Tomonori
same as of10. Signed-off-by: FUJITA Tomonori --- ryu/ofproto/ofproto_v1_3_parser.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ryu/ofproto/ofproto_v1_3_parser.py b/ryu/ofproto/ofproto_v1_3_parser.py index 0d38dee..00dd881 100644 --- a/ryu/ofproto/ofproto_v1_3_parser.py

Re: [Ryu-devel] Link Detection problem

2013-08-22 Thread FUJITA Tomonori
On Thu, 22 Aug 2013 16:36:15 +0900 (JST) FUJITA Tomonori wrote: > On Wed, 21 Aug 2013 13:01:01 -0300 > Tonhão - Antonio Brasolin wrote: > >> I'm trying to run an app that detects links but, for some reason, the link >> event just isn't happening. Is it some change on an API? It works fine on >>

Re: [Ryu-devel] [PATCH 01/16] of10: use text address for OFPPhyPort.hw_addr and OFPPortMod.hw_addr

2013-08-22 Thread FUJITA Tomonori
On Fri, 16 Aug 2013 15:29:22 +0900 YAMAMOTO Takashi wrote: > Signed-off-by: YAMAMOTO Takashi > --- > ryu/ofproto/ofproto_v1_0_parser.py | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/ryu/ofproto/ofproto_v1_0_parser.py > b/ryu/ofproto/ofproto_v1_0_parser.py > ind

[Ryu-devel] [PATCH] packet lib: fix ipv6 parser

2013-08-22 Thread FUJITA Tomonori
From: Ygor Amaral payload length in ipv6 header doesn't include the basic ipv6 header length (40 bytes). Signed-off-by: FUJITA Tomonori --- ryu/lib/packet/ipv6.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ryu/lib/packet/ipv6.py b/ryu/lib/packet/ipv6.py index b65b823

Re: [Ryu-devel] Link Detection problem

2013-08-22 Thread FUJITA Tomonori
On Wed, 21 Aug 2013 13:01:01 -0300 Tonhão - Antonio Brasolin wrote: > I'm trying to run an app that detects links but, for some reason, the link > event just isn't happening. Is it some change on an API? It works fine on > older versions of ryu and on the version with the GUI. > > Thank you in a