[Ryu-devel] [PATCH 05/13] log-cleanup: per RyuApp logger

2013-03-04 Thread YAMAMOTO Takashi
let RyuApp provide its own logger. use it instead of per-module logger where appropriate. Signed-off-by: YAMAMOTO Takashi --- ryu/app/gre_tunnel.py | 15 -- ryu/app/quantum_adapter.py | 65 ryu/app/simple_isolation.py| 42

[Ryu-devel] [PATCH 09/13] cli: add a utility module

2013-03-04 Thread YAMAMOTO Takashi
this will be used by cli app. Signed-off-by: YAMAMOTO Takashi --- ryu/plogger.py | 31 +++ 1 file changed, 31 insertions(+) create mode 100644 ryu/plogger.py diff --git a/ryu/plogger.py b/ryu/plogger.py new file mode 100644 index 000..c95a461 --- /dev/null +++ b

[Ryu-devel] [PATCH 12/13] cli: completion of logger names for set_log_level command

2013-03-04 Thread YAMAMOTO Takashi
Signed-off-by: YAMAMOTO Takashi --- ryu/app/cli.py | 6 ++ 1 file changed, 6 insertions(+) diff --git a/ryu/app/cli.py b/ryu/app/cli.py index 8bdf1dd..36df047 100644 --- a/ryu/app/cli.py +++ b/ryu/app/cli.py @@ -91,6 +91,12 @@ class CliCmd(cmd.Cmd): return print('logger

[Ryu-devel] [PATCH 11/13] cli: add cli app

2013-03-04 Thread YAMAMOTO Takashi
Signed-off-by: YAMAMOTO Takashi --- ryu/app/cli.py | 227 + 1 file changed, 227 insertions(+) create mode 100644 ryu/app/cli.py diff --git a/ryu/app/cli.py b/ryu/app/cli.py new file mode 100644 index 000..8bdf1dd --- /dev/null +++ b/ry

[Ryu-devel] [PATCH 06/13] log-cleanup: honor --default-log-level in case of --verbose

2013-03-04 Thread YAMAMOTO Takashi
if the default log level is explicitly specified by --default-log-level option, use the level rather than defaulting to logging.DEBUG. Signed-off-by: YAMAMOTO Takashi --- ryu/log.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ryu/log.py b/ryu/log.py index db88ee7..d

[Ryu-devel] [PATCH 03/13] log-cleanup: remove an unused logger instance

2013-03-04 Thread YAMAMOTO Takashi
Signed-off-by: YAMAMOTO Takashi --- ryu/controller/tunnels.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/ryu/controller/tunnels.py b/ryu/controller/tunnels.py index e322d3a..0a94673 100644 --- a/ryu/controller/tunnels.py +++ b/ryu/controller/tunnels.py @@ -22,9 +22,6 @@ from ryu.base i

[Ryu-devel] [PATCH 04/13] log-cleanup: remove an unused logger instance

2013-03-04 Thread YAMAMOTO Takashi
Signed-off-by: YAMAMOTO Takashi --- ryu/app/rest_conf_switch.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/ryu/app/rest_conf_switch.py b/ryu/app/rest_conf_switch.py index 0cd1541..57e2347 100644 --- a/ryu/app/rest_conf_switch.py +++ b/ryu/app/rest_conf_switch.py @@ -25,9 +25,6 @@ from

[Ryu-devel] [PATCH 08/13] cli: a simple management api

2013-03-04 Thread YAMAMOTO Takashi
this will be used by cli app. Signed-off-by: YAMAMOTO Takashi --- ryu/base/management.py | 57 ++ 1 file changed, 57 insertions(+) create mode 100644 ryu/base/management.py diff --git a/ryu/base/management.py b/ryu/base/management.py new file mod

[Ryu-devel] [PATCH 13/13] cli: sample config

2013-03-04 Thread YAMAMOTO Takashi
--- etc/ryu/ryu.conf | 5 + 1 file changed, 5 insertions(+) diff --git a/etc/ryu/ryu.conf b/etc/ryu/ryu.conf index 341de67..702492e 100644 --- a/etc/ryu/ryu.conf +++ b/etc/ryu/ryu.conf @@ -6,3 +6,8 @@ #wsapi_port= #ofp_listen_host= #ofp_listen_port= + +cli_transports=ssh +cli_ssh_username=

[Ryu-devel] [PATCH 10/13] cli: add a utility module

2013-03-04 Thread YAMAMOTO Takashi
this will be used by cli app. Signed-off-by: YAMAMOTO Takashi --- ryu/call_via_pipe.py | 100 +++ 1 file changed, 100 insertions(+) create mode 100644 ryu/call_via_pipe.py diff --git a/ryu/call_via_pipe.py b/ryu/call_via_pipe.py new file mode 100

[Ryu-devel] [PATCH 00/13] cli app and log cleanup

2013-03-04 Thread YAMAMOTO Takashi
this contains: - management api - cli app - log cleanup YAMAMOTO Takashi (13): ryu-manager: stop modifying CONF.app_list log-cleanup: comment out an unused variable log-cleanup: remove an unused logger instance log-cleanup: remove an unused logger instance log-cle

[Ryu-devel] [PATCH 07/13] maintain a list of loggers

2013-03-04 Thread YAMAMOTO Takashi
this will be used by management api. Signed-off-by: YAMAMOTO Takashi --- bin/ryu-manager | 1 + ryu/logger.py | 31 +++ 2 files changed, 32 insertions(+) create mode 100644 ryu/logger.py diff --git a/bin/ryu-manager b/bin/ryu-manager index 03b1b79..18a38d6 100755

[Ryu-devel] [PATCH 01/13] ryu-manager: stop modifying CONF.app_list

2013-03-04 Thread YAMAMOTO Takashi
to make cli show-options a little prettier. Signed-off-by: YAMAMOTO Takashi --- bin/ryu-manager | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/bin/ryu-manager b/bin/ryu-manager index f154bfe..03b1b79 100755 --- a/bin/ryu-manager +++ b/bin/ryu-manager @@ -58,11 +58,10 @@

[Ryu-devel] [PATCH 02/13] log-cleanup: comment out an unused variable

2013-03-04 Thread YAMAMOTO Takashi
because all of its possible users are commented out. Signed-off-by: YAMAMOTO Takashi --- ryu/app/wsgi.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ryu/app/wsgi.py b/ryu/app/wsgi.py index 3779b08..68a7b28 100644 --- a/ryu/app/wsgi.py +++ b/ryu/app/wsgi.py @@ -22,7 +22,7

[Ryu-devel] [PATCH] ryu.log: change type of --use_syslog option to Boolean.

2013-03-04 Thread EisukeTAKAHASHI
-- $ ryu-manager --help () --use-syslog USE_SYSLOG output to syslog $ ryu-manager --use-syslog usage: ryu-manager [-h] [--version] [--verbose] [--noverbose] [--use-stderr] () ryu-manager: error: argument --use-syslog: expected one argument -- Sign

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

2013-03-04 Thread Isaku Yamahata
On Tue, Mar 05, 2013 at 07:34:27AM +0900, FUJITA Tomonori wrote: > On Fri, 22 Feb 2013 19:14:19 +0900 > YAMADA Hideki wrote: > > > Now, direct method calling is used for app-to-app communication, > > via _CONTEXTS parameters. > > This patch is more loose coupling way than direct way. > > > > Sig

Re: [Ryu-devel] [PATCH] contrib: import ncclient library (NETCONF clients)

2013-03-04 Thread FUJITA Tomonori
On Tue, 5 Mar 2013 06:37:12 +0900 Isaku Yamahata wrote: > On Tue, Mar 05, 2013 at 06:28:03AM +0900, FUJITA Tomonori wrote: >> On Tue, 5 Mar 2013 06:26:39 +0900 >> Isaku Yamahata wrote: >> >> > On Mon, Mar 04, 2013 at 11:05:27PM +0900, FUJITA Tomonori wrote: >> >> On Mon, 4 Mar 2013 14:14:55 +09

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

2013-03-04 Thread FUJITA Tomonori
On Fri, 22 Feb 2013 19:14:19 +0900 YAMADA Hideki wrote: > 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 ++

Re: [Ryu-devel] [PATCH] contrib: import ncclient library (NETCONF clients)

2013-03-04 Thread Isaku Yamahata
On Tue, Mar 05, 2013 at 06:28:03AM +0900, FUJITA Tomonori wrote: > On Tue, 5 Mar 2013 06:26:39 +0900 > Isaku Yamahata wrote: > > > On Mon, Mar 04, 2013 at 11:05:27PM +0900, FUJITA Tomonori wrote: > >> On Mon, 4 Mar 2013 14:14:55 +0900 > >> Isaku Yamahata wrote: > >> > >> > On Mon, Mar 04, 2013

Re: [Ryu-devel] [PATCH] contrib: import ncclient library (NETCONF clients)

2013-03-04 Thread FUJITA Tomonori
On Tue, 5 Mar 2013 06:26:39 +0900 Isaku Yamahata wrote: > On Mon, Mar 04, 2013 at 11:05:27PM +0900, FUJITA Tomonori wrote: >> On Mon, 4 Mar 2013 14:14:55 +0900 >> Isaku Yamahata wrote: >> >> > On Mon, Mar 04, 2013 at 02:01:05PM +0900, FUJITA Tomonori wrote: >> >> On Mon, 4 Mar 2013 13:57:42 +09

Re: [Ryu-devel] [PATCH] contrib: import ncclient library (NETCONF clients)

2013-03-04 Thread Isaku Yamahata
On Mon, Mar 04, 2013 at 11:05:27PM +0900, FUJITA Tomonori wrote: > On Mon, 4 Mar 2013 14:14:55 +0900 > Isaku Yamahata wrote: > > > On Mon, Mar 04, 2013 at 02:01:05PM +0900, FUJITA Tomonori wrote: > >> On Mon, 4 Mar 2013 13:57:42 +0900 > >> Isaku Yamahata wrote: > >> > >> > I needed the followin

Re: [Ryu-devel] match fields of incoming packet

2013-03-04 Thread FUJITA Tomonori
On Mon, 4 Mar 2013 15:26:14 + "Henkel, Michael" wrote: > I'm only getting the in_port field back. I also would like to get the > eth_src/dst/type fields but they don't seem to exist. > I am sending an arp packet, looping through the packet everything is there, > it's just that I'd like to r

Re: [Ryu-devel] match fields of incoming packet

2013-03-04 Thread Henkel, Michael
I'm only getting the in_port field back. I also would like to get the eth_src/dst/type fields but they don't seem to exist. I am sending an arp packet, looping through the packet everything is there, it's just that I'd like to read the header fields instead of looping like this: for p in

Re: [Ryu-devel] match fields of incoming packet

2013-03-04 Thread FUJITA Tomonori
On Mon, 4 Mar 2013 12:52:02 + "Henkel, Michael" wrote: > Using OF 1.2 I'm trying to match different header fields of incoming packets. > Unfortunately on the in_port field seems to exist: Sorry, I'm not sure what you mean. Your code looks fine. What's the problem? -

Re: [Ryu-devel] [PATCH] contrib: import ncclient library (NETCONF clients)

2013-03-04 Thread FUJITA Tomonori
On Mon, 4 Mar 2013 14:14:55 +0900 Isaku Yamahata wrote: > On Mon, Mar 04, 2013 at 02:01:05PM +0900, FUJITA Tomonori wrote: >> On Mon, 4 Mar 2013 13:57:42 +0900 >> Isaku Yamahata wrote: >> >> > I needed the following patch to make it work with LINC switch. >> >> Thanks, can you try to push it t

[Ryu-devel] match fields of incoming packet

2013-03-04 Thread Henkel, Michael
Hi list, Using OF 1.2 I'm trying to match different header fields of incoming packets. Unfortunately on the in_port field seems to exist: Code: @set_ev_cls(ofp_event.EventOFPPacketIn, MAIN_DISPATCHER) def packet_in_handler(self, ev): msg = ev.msg dp = msg.datapath

Re: [Ryu-devel] [PATCH] fix a comment typo

2013-03-04 Thread FUJITA Tomonori
On Fri, 1 Mar 2013 17:29:52 +0900 YAMAMOTO Takashi wrote: > Signed-off-by: YAMAMOTO Takashi > --- > ryu/ofproto/ofproto_v1_3.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Applied, thanks. -- Everyone hat

Re: [Ryu-devel] [PATCH] revert s/_/-/ changes for ryu.conf

2013-03-04 Thread FUJITA Tomonori
On Mon, 4 Mar 2013 21:01:32 +0900 Isaku Yamahata wrote: > On Mon, Mar 04, 2013 at 08:32:16PM +0900, FUJITA Tomonori wrote: >> On Mon, 4 Mar 2013 20:30:17 +0900 (JST) >> [email protected] (YAMAMOTO Takashi) wrote: >> >> >> On Mon, 4 Mar 2013 15:58:07 +0900 (JST) >> >> [email protected]

Re: [Ryu-devel] [PATCH] revert s/_/-/ changes for ryu.conf

2013-03-04 Thread Isaku Yamahata
On Mon, Mar 04, 2013 at 08:32:16PM +0900, FUJITA Tomonori wrote: > On Mon, 4 Mar 2013 20:30:17 +0900 (JST) > [email protected] (YAMAMOTO Takashi) wrote: > > >> On Mon, 4 Mar 2013 15:58:07 +0900 (JST) > >> [email protected] (YAMAMOTO Takashi) wrote: > >> > >>> hi, > >>> > On Fri,

Re: [Ryu-devel] [PATCH] revert s/_/-/ changes for ryu.conf

2013-03-04 Thread FUJITA Tomonori
On Mon, 4 Mar 2013 20:30:17 +0900 (JST) [email protected] (YAMAMOTO Takashi) wrote: >> On Mon, 4 Mar 2013 15:58:07 +0900 (JST) >> [email protected] (YAMAMOTO Takashi) wrote: >> >>> hi, >>> On Fri, 1 Mar 2013 14:13:47 +0900 YAMAMOTO Takashi wrote: > cfg.py automa

Re: [Ryu-devel] [PATCH] revert s/_/-/ changes for ryu.conf

2013-03-04 Thread YAMAMOTO Takashi
> On Mon, 4 Mar 2013 15:58:07 +0900 (JST) > [email protected] (YAMAMOTO Takashi) wrote: > >> hi, >> >>> On Fri, 1 Mar 2013 14:13:47 +0900 >>> YAMAMOTO Takashi wrote: >>> cfg.py automatically replaces '-' in option names to '_' for config files. so s/_/-/ in this file was a mist

Re: [Ryu-devel] [PATCH] revert s/_/-/ changes for ryu.conf

2013-03-04 Thread FUJITA Tomonori
On Mon, 4 Mar 2013 15:58:07 +0900 (JST) [email protected] (YAMAMOTO Takashi) wrote: > hi, > >> On Fri, 1 Mar 2013 14:13:47 +0900 >> YAMAMOTO Takashi wrote: >> >>> cfg.py automatically replaces '-' in option names to '_' for config files. >>> so s/_/-/ in this file was a mistake. revert