Re: [Ryu-devel] [PATCH v3 3/3] ofctl: Add some tests for ofp_instruction_from_str

2017-05-23 Thread Iwase Yusuke
Hi, On 2017年05月23日 16:05, IWAMOTO Toshihiro wrote: > Signed-off-by: IWAMOTO Toshihiro > --- > ryu/tests/unit/lib/test_ofctl_string.py | 157 > > 1 file changed, 157 insertions(+) > create mode 100644 ryu/tests/unit/lib/test_ofctl_string.py > > diff --git a/ry

Re: [Ryu-devel] [PATCH v3 2/3] ofctl: Add ovs-ofctl style action string parser

2017-05-23 Thread Iwase Yusuke
Hi, On 2017年05月23日 16:05, IWAMOTO Toshihiro wrote: > diff --git a/ryu/lib/ofctl_string.py b/ryu/lib/ofctl_string.py > ...(snip) > + > +import re > + > +import ryu.exception > +from ryu.lib.ofctl_utils import str_to_int > +from ryu.ofproto import nicira_ext > + For pep8 happy, 2 blank lines expec

Re: [Ryu-devel] Running subset of Ryu tests

2017-05-23 Thread Fujimoto Satoshi
Hi, Lumir I expect you run tests with ./run_tests.sh. If you don't have Docker, the script automatically skips tests which need Docker, so you don't have to care about it. However, there is one test which needs Internet connection, and the way to skip the test is not provided. As a result, ./ru

Re: [Ryu-devel] Learn Action

2017-05-23 Thread Iwase Yusuke
Hi Salaheddine, On 2017年05月23日 19:43, Salaheddine ZERKANE wrote: > Hello Ryu community, > > Could you please tell me if Ryu implement “Learn” action found in OpenVswitch? Of course! Please refer to the following for the usages. http://ryu.readthedocs.io/en/latest/nicira_ext_ref.html#ryu.ofpr

[Ryu-devel] Running subset of Ryu tests

2017-05-23 Thread Lumir Balhar
Hello. I am working on python-ryu RPM package in Fedora and I want to keep them up to date as possible but I have one problem with tests. I want to enable tests during build of RPM package, but I cannot run all of them because (for example): - build system cannot run docker containers - build s

[Ryu-devel] Learn Action

2017-05-23 Thread Salaheddine ZERKANE
Hello Ryu community, Could you please tell me if Ryu implement "Learn" action found in OpenVswitch? If not, how can we implement it in Ryu? Thanks and regards, Meilleures salutations, Salaheddine Zerkane {P} Ingénieur doctorant Cubiq {T} +33 (0) 2 56 35 82 11 [http://signature.b-com.com/logo-

[Ryu-devel] [PATCH v3 3/3] ofctl: Add some tests for ofp_instruction_from_str

2017-05-23 Thread IWAMOTO Toshihiro
Signed-off-by: IWAMOTO Toshihiro --- ryu/tests/unit/lib/test_ofctl_string.py | 157 1 file changed, 157 insertions(+) create mode 100644 ryu/tests/unit/lib/test_ofctl_string.py diff --git a/ryu/tests/unit/lib/test_ofctl_string.py b/ryu/tests/unit/lib/test_ofctl

[Ryu-devel] [PATCH v3 0/3] Add ovs-ofctl style action string parser

2017-05-23 Thread IWAMOTO Toshihiro
This set of patches allow users to use ovs-ofctl style action strings to build OFPInstructionActions, which may help transition from ovs-ofctl. Support is currently very limited and many methods should be added to OfctlActionConverter for this to be useful. v2 -> v3: - Moved action string parser

[Ryu-devel] [PATCH v3 1/3] nicira_ext: Define some CT constants

2017-05-23 Thread IWAMOTO Toshihiro
Signed-off-by: IWAMOTO Toshihiro --- ryu/ofproto/nicira_ext.py | 5 + 1 file changed, 5 insertions(+) diff --git a/ryu/ofproto/nicira_ext.py b/ryu/ofproto/nicira_ext.py index 411ed48..416c625 100644 --- a/ryu/ofproto/nicira_ext.py +++ b/ryu/ofproto/nicira_ext.py @@ -269,6 +269,11 @@ NX_LEARN

[Ryu-devel] [PATCH v3 2/3] ofctl: Add ovs-ofctl style action string parser

2017-05-23 Thread IWAMOTO Toshihiro
This commit adds a new method called ofp_instruction_from_str, which takes an ovs-ofctl style action string and returns a list of JSON dictionaries. Currently only a few action strings are understood. Signed-off-by: IWAMOTO Toshihiro --- ryu/exception.py | 4 + ryu/lib/ofctl_strin