Re: [Ryu-devel] [PATCH v2] manager: Except KeyboardInterrupt to suppress traceback

2016-07-26 Thread FUJITA Tomonori
On Wed, 27 Jul 2016 14:55:26 +0900 IWASE Yusuke wrote: > Currently, ryu-manager emits traceback messages when killed via > KeyboardInterrupt. > This patch traps the KeyboardInterrupt, and outputs an appropriate > log message instead. > > Signed-off-by: Victor J. Orlikowski > Signed-off-by: Alan

Re: [Ryu-devel] [PATCH 0/2] stplib: support other switch implementation

2016-07-26 Thread FUJITA Tomonori
On Mon, 25 Jul 2016 14:41:05 +0900 IWASE Yusuke wrote: > stplib.py is used in simple_switch_stp_13.py which is an example > on Ryu-Book and supposes OVS as the connected switch. > So stplib is implemented with some assumptions for OVS behaviors > and in some cases, it cannot connect to other swit

Re: [Ryu-devel] [PATCH] rest_qos: Fix to handle the case without QoS action

2016-07-26 Thread FUJITA Tomonori
On Thu, 21 Jul 2016 13:33:20 +0900 IWASE Yusuke wrote: > Currently, rest_qos.py might fail to parse the actions field in > the QoS rules when the actions field does not have any actions > for the QoS (SET_FIELD, METER, SET_QUEUE). > > This patch fixes this problem. > > Signed-off-by: IWASE Yusu

[Ryu-devel] [PATCH v2] manager: Except KeyboardInterrupt to suppress traceback

2016-07-26 Thread IWASE Yusuke
Currently, ryu-manager emits traceback messages when killed via KeyboardInterrupt. This patch traps the KeyboardInterrupt, and outputs an appropriate log message instead. Signed-off-by: Victor J. Orlikowski Signed-off-by: Alan Fordyce Signed-off-by: IWASE Yusuke --- ryu/cmd/manager.py | 9

Re: [Ryu-devel] Patch: Support relayed DHCP packets

2016-07-26 Thread FUJITA Tomonori
On Tue, 19 Jul 2016 20:13:31 -0700 Michael Haro wrote: > https://tools.ietf.org/html/draft-ietf-dhc-implementation-02#section-4.7.2 > > states that DHCP relays should use source port 67 and dest port 67. > > --- > ryu/lib/packet/udp.py | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-)

Re: [Ryu-devel] KeyboardInterrupt (ctrl-c) and RYU

2016-07-26 Thread Iwase Yusuke
Hi Fujita-San, On 2016年07月27日 14:35, FUJITA Tomonori wrote: > On Thu, 21 Jul 2016 16:41:03 +0900 > Iwase Yusuke wrote: > >> Hi Alan, >> >> Thank you for reviewing my patch! > > Can you please send the patch in the proper format? Sure, I will send this patch again. Regards, Iwase -

Re: [Ryu-devel] Query regarding learning flow configuration

2016-07-26 Thread Shinpei Muraoka
Hi > This learning flow is not getting configured on the switch due to command syntax error. > There is some mistake with the command syntax. > > Could you please send me an example of a learning flow similar to what i have done ? or is it possible to point out the mistake in above syntax

Re: [Ryu-devel] KeyboardInterrupt (ctrl-c) and RYU

2016-07-26 Thread FUJITA Tomonori
On Thu, 21 Jul 2016 16:41:03 +0900 Iwase Yusuke wrote: > Hi Alan, > > Thank you for reviewing my patch! Can you please send the patch in the proper format? -- What NetFlow Analyzer can do for you? Monitors network band

[Ryu-devel] [PATCH v4 2/2] ofproto/nx_actions: Update argument of NXActionCT

2016-07-26 Thread Shinpei Muraoka
Since zone_src in NXActionCT was not possible to specify string, update zone_src of NXActionCT for the uniformity. Therefore, you will be able to specify string of OXM/NXM fields for zone_src. If you want to set the immediate value for zone, zone_src set the None or empty character string. Signed

[Ryu-devel] [PATCH v4 1/2] ofproto/nx_actions: Revert API of NXAction

2016-07-26 Thread Shinpei Muraoka
This patch will revert the API of NXAction for the following report. https://bugs.launchpad.net/dragonflow/+bug/1599367 revert commit id: 3c3896fd77f274ce953dcb78795aa5cc4fde6c65 This patch reverts recently introduced start and end arguments. Add the utility method for ofs_nbits. NXActionRegLoad n

[Ryu-devel] [PATCH v4 0/2] Revert API of NXAction

2016-07-26 Thread Shinpei Muraoka
Also, the results of discussions with Fujita-san and Iwamoto-San, the type of argument zone_src of NXActionCT was changed to a string. Fix to work even if you specify a number in the NXActionCT of argument zone_src. Argument of NXActionRegLoad is the format you use the ofs_nbits. Add a commit mess

Re: [Ryu-devel] [PATCH v3 1/2] ofproto/nx_actions: Revert API of NXAction

2016-07-26 Thread Shinpei Muraoka
Hi Fujita-San Iwamoto-San As you indicated, Add to commit message on modification of NXActionRegLoad. This patch will revert the API of NXAction for the following report. https://bugs.launchpad.net/dragonflow/+bug/1599367 revert commit id: 3c3896fd77f274ce953dcb78795aa5cc4fde6c65

Re: [Ryu-devel] [PATCH v3 2/2] ofproto/nx_actions: Update argument of NXActionCT

2016-07-26 Thread Shinpei Muraoka
Hi Fujita-San > With the current path, still Dragonflow developers need to change > their code? If so, Muraoka-san, please fix the patch. If not, I'll > push this patch. > For now, this patch does not support the number of non-zero. Therefore, to fix the patch. Also, add the following senten

Re: [Ryu-devel] [PATCH v3 1/2] ofproto/nx_actions: Revert API of NXAction

2016-07-26 Thread IWAMOTO Toshihiro
At Wed, 27 Jul 2016 07:02:29 +0900 (JST), FUJITA Tomonori wrote: > > Iwamoto-san, any comment on this? It might be helpful to mention - NXActionRegLoad now takes ofs_nbits argument instead of ofs and nbits even with this revert and being a bit explicit about what this patch reverts: -

Re: [Ryu-devel] [PATCH v3 1/2] ofproto/nx_actions: Revert API of NXAction

2016-07-26 Thread FUJITA Tomonori
Iwamoto-san, any comment on this? On Fri, 22 Jul 2016 13:59:48 +0900 Shinpei Muraoka wrote: > This patch will revert the API of NXAction for the following report. > https://bugs.launchpad.net/dragonflow/+bug/1599367 > revert commit id: 3c3896fd77f274ce953dcb78795aa5cc4fde6c65 > > Add the utilit

Re: [Ryu-devel] [PATCH v3 2/2] ofproto/nx_actions: Update argument of NXActionCT

2016-07-26 Thread FUJITA Tomonori
On Tue, 26 Jul 2016 17:05:52 +0900 IWAMOTO Toshihiro wrote: > At Tue, 26 Jul 2016 16:25:15 +0900, > Shinpei Muraoka wrote: >> >> Hi Iwamoto-San >> >> >> On 2016年07月26日 13:55, IWAMOTO Toshihiro wrote: >> > Thanks for cleaning up API incoherency. >> > >> > At Fri, 22 Jul 2016 13:59:49 +0900, >>

[Ryu-devel] Query regarding learning flow configuration

2016-07-26 Thread Krishnamurthy Karthika
Hi Ryu Developers, Could you please help me with the following REST API query ? I need to configure a new learning flow on the switch through a new REST API with given input parameters. I took the json example given in the following path https://github.com/osrg/ryu/blob/a3c8c591d9e6fe1353f7ba5

[Ryu-devel] Ryu for teaching networking

2016-07-26 Thread Greg Bernstein
Hi all, I have started to get ready for teaching networking in the Fall (including a few weeks on SDN) by expanding some of my teaching materials to include Ryu . See this page a walk through an example “o

Re: [Ryu-devel] [PATCH v3 2/2] ofproto/nx_actions: Update argument of NXActionCT

2016-07-26 Thread IWAMOTO Toshihiro
At Tue, 26 Jul 2016 16:25:15 +0900, Shinpei Muraoka wrote: > > Hi Iwamoto-San > > > On 2016年07月26日 13:55, IWAMOTO Toshihiro wrote: > > Thanks for cleaning up API incoherency. > > > > At Fri, 22 Jul 2016 13:59:49 +0900, > > Shinpei Muraoka wrote: > >> > >> Since zone_src in NXActionCT was not p

Re: [Ryu-devel] [PATCH v3 2/2] ofproto/nx_actions: Update argument of NXActionCT

2016-07-26 Thread Shinpei Muraoka
Hi Iwamoto-San On 2016年07月26日 13:55, IWAMOTO Toshihiro wrote: > Thanks for cleaning up API incoherency. > > At Fri, 22 Jul 2016 13:59:49 +0900, > Shinpei Muraoka wrote: >> >> Since zone_src in NXActionCT was not possible to specify string, >> update zone_src of NXActionCT for the uniformity. >> T