Hi Iwase-san,

Thank you for reviewing.

On Tue, 1 Nov 2016 14:07:04 +0900
Iwase Yusuke <[email protected]> wrote:

> Hi Kakuma-San,
> 
> Docker service is also required to run unit tests in my patch.
> Please refer to the following post.
>    [PATCH v2] ut/ovs: Add UT using Docker for ryu.lib.ovs

Sorry, I will consider it.

> 
> BTW, I prefer to switching the version in tox.ini file as following?
> What do you think?
> 

Oh, it seems to be more simple. I will update it like that.

> ... (snip)
> 
> [testenv]
> deps = -U
>         -r{toxinidir}/tools/pip-requires
>         -r{toxinidir}/tools/test-requires
>         --no-cache-dir
> usedevelop = True
> passenv= NOSE_VERBOSE
> commands =
>    coverage run --source=ryu ryu/tests/run_tests.py '{posargs}'
> 
> [testenv:py27]
> deps = -U
>         -r{toxinidir}/tools/pip-requires
>         -r{toxinidir}/tools/test-requires
>         --no-cache-dir
> usedevelop = True
> passenv= NOSE_VERBOSE
> commands =
>    coverage run --source=ryu ryu/tests/run_tests.py '{posargs}'
>    bash ryu/tests/integrated/common/install_docker_test_pkg_for_travis.sh
>    python ryu/tests/integrated/run_test.py
> 
> ... (snip)
> 
> 
> Thanks,
> Iwase
> 
> On 2016年11月01日 12:09, fumihiko kakuma wrote:
> > Signed-off-by: Fumihiko Kakuma <[email protected]>
> > ---
> >  .travis.yml | 43 ++++++++++++++++++++++++++++---------------
> >  tox.ini     |  3 +++
> >  2 files changed, 31 insertions(+), 15 deletions(-)
> >
> > diff --git a/.travis.yml b/.travis.yml
> > index 82d194f..51bde54 100644
> > --- a/.travis.yml
> > +++ b/.travis.yml
> > @@ -1,23 +1,36 @@
> >  language: python
> > -python:
> > -  - "3.5"  # Python 3.5 still needs to be installed on Travis-CI
> >
> > -services:
> > -  - docker
> > -
> > -env:
> > -  - TOX_ENV=py27
> > -  - TOX_ENV=py34
> > -  - TOX_ENV=py35
> > -  - TOX_ENV=pypy26
> > -  - TOX_ENV=pep8
> > +matrix:
> > +  include:
> > +  - env: TOX_ENV=py27
> > +    python: "2.7"
> > +    services: docker
> > +    sudo: required
> > +    script:
> > +    - NOSE_VERBOSE=0 tox -e $TOX_ENV
> > +    - tox -e scenario
> > +  - env: TOX_ENV=py34
> > +    python: "3.4"
> > +    services: docker
> > +    sudo: required
> > +    script:
> > +    - NOSE_VERBOSE=0 tox -e $TOX_ENV
> > +    - tox -e scenario
> > +  - env: TOX_ENV=py35
> > +    python: "3.5"
> > +    script:
> > +    - NOSE_VERBOSE=0 tox -e $TOX_ENV
> > +  - env: TOX_ENV=pypy26
> > +    python: "3.5"
> > +    script:
> > +    - NOSE_VERBOSE=0 tox -e $TOX_ENV
> > +  - env: TOX_ENV=pep8
> > +    python: "3.5"
> > +    script:
> > +    - NOSE_VERBOSE=0 tox -e $TOX_ENV
> >
> >  install:
> >    - pip install tox coveralls
> >
> > -sudo: required
> > -script:
> > -  - NOSE_VERBOSE=0 tox -e $TOX_ENV
> > -
> >  after_success:
> >    - coveralls
> > diff --git a/tox.ini b/tox.ini
> > index ae9fd33..964c02c 100644
> > --- a/tox.ini
> > +++ b/tox.ini
> > @@ -10,6 +10,9 @@ usedevelop = True
> >  passenv= NOSE_VERBOSE
> >  commands =
> >    coverage run --source=ryu ryu/tests/run_tests.py '{posargs}'
> > +
> > +[testenv:scenario]
> > +commands =
> >    bash ryu/tests/integrated/common/install_docker_test_pkg_for_travis.sh
> >    python ryu/tests/integrated/run_test.py
> >
> >

-- 
fumihiko kakuma <[email protected]>



------------------------------------------------------------------------------
Developer Access Program for Intel Xeon Phi Processors
Access to Intel Xeon Phi processor-based developer platforms.
With one year of Intel Parallel Studio XE.
Training and support from Colfax.
Order your platform today. http://sdm.link/xeonphi
_______________________________________________
Ryu-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ryu-devel

Reply via email to