cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sebastien Goasguen" <run...@gmail.com>
Subject Re: Review Request 23856: Fix VPC distributed routing integration tests
Date Mon, 06 Apr 2015 20:06:52 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23856/#review79030
-----------------------------------------------------------


Thank you for submitting your CloudStack contribution through review board. After discussion
on the dev@cloudstack.apache.org the community decided to close down review board and start
accepting contributiong through GitHub pull requests. We have been using GH PR for several
months now and the process is better than review board.

We will keep Review Board open for another week to give you time to migrate your patch to
a github PR if you wish. After that time, your patch will no longer be viewable (even though
it will not be deleted).

Please consider submitting a pull request.

Great instructions are available at:
https://github.com/apache/cloudstack/blob/master/CONTRIBUTING.md

Thank you very much for your time and your contribution to Apache CloudStack, we hope that
using this new process will encourage you to do more.

- Sebastien Goasguen


On July 24, 2014, 1:46 p.m., Miguel Ferreira wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/23856/
> -----------------------------------------------------------
> 
> (Updated July 24, 2014, 1:46 p.m.)
> 
> 
> Review request for cloudstack, daan Hoogland, Murali Reddy, Hugo Trippaers, and SrikanteswaraRao
Talluri.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> 1) The first test in the class is failing on asserting that distributed routing is enabled:
> That was because it is using the global services configuration instead of the local one
(where the distributed routing is enabled).
> I've changed that and also added two new lines in between every python method to make
the class more readable.
> 
> See stack trace:
> Traceback (most recent call last):
>   File "/usr/local/Cellar/python/2.7.8/Frameworks/Python.framework/Versions/2.7/lib/python2.7/unittest/case.py",
line 329, in run
>     testMethod()
>   File "/Users/mferreira/development/workspace/cloudstack-sbp-vpc-tests/src/vpc-tests/all/test_vpc_distributed_routing_offering.py",
line 292, in test_01_create_vpc_offering_with_distributedrouter_service_capability
>     self.validate_vpc_offering(vpc_off)
>   File "/Users/mferreira/development/workspace/cloudstack-sbp-vpc-tests/src/vpc-tests/all/test_vpc_distributed_routing_offering.py",
line 245, in validate_vpc_offering
>     "VPC offering is not set up for Distributed routing"
>   File "/usr/local/Cellar/python/2.7.8/Frameworks/Python.framework/Versions/2.7/lib/python2.7/unittest/case.py",
line 513, in assertEqual
>     assertion_func(first, second, msg=msg)
>   File "/usr/local/Cellar/python/2.7.8/Frameworks/Python.framework/Versions/2.7/lib/python2.7/unittest/case.py",
line 506, in _baseAssertEqual
>     raise self.failureException(msg)
> AssertionError: VPC offering is not set up for Distributed routing
> -------------------- >> begin captured stdout << ---------------------
> === TestName: test_01_create_vpc_offering_with_distributedrouter_service_capability |
Status : FAILED ===
> 
> 
> 2) The second test was failing because the connectivity service was not being enabled
in te offering due to a typo.
> 
> See stack trace:
> Traceback (most recent call last):
>   File "/usr/local/Cellar/python/2.7.8/Frameworks/Python.framework/Versions/2.7/lib/python2.7/unittest/case.py",
line 329, in run
>     testMethod()
>   File "/Users/mferreira/development/workspace/cloudstack-sbp-vpc-tests/src/vpc-tests/all/test_vpc_distributed_routing_offering.py",
line 369, in test_03_deploy_vms_in_vpc_with_distributedrouter
>     conservemode=False
>   File "/Users/mferreira/development/git/cloudstack-sbp/tools/marvin/marvin/lib/base.py",
line 2028, in create
>     return NetworkOffering(apiclient.createNetworkOffering(cmd).__dict__)
>   File "/Users/mferreira/development/git/cloudstack-sbp/tools/marvin/marvin/cloudstackAPI/cloudstackAPIClient.py",
line 1778, in createNetworkOffering
>     response = self.connection.marvinRequest(command, response_type=response, method=method)
>   File "/Users/mferreira/development/git/cloudstack-sbp/tools/marvin/marvin/cloudstackConnection.py",
line 382, in marvinRequest
>     raise e
> CloudstackAPIException: Execute cmd: createnetworkoffering failed, due to: errorCode:
431, errorText:Invalid service  Connectivity
> 
> 
> 3) The tests require that the Ovs network provider be enabled, but it is disabled by
default when created. I've added a method to the setup that makes sure Ovs is enabled, or
throws an exception if it is not present.
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_vpc_distributed_routing_offering.py cc9a191 
>   tools/marvin/marvin/lib/common.py 187ede6 
> 
> Diff: https://reviews.apache.org/r/23856/diff/
> 
> 
> Testing
> -------
> 
> Both test #1 and #2 in that class succeed.
> 
> 
> Thanks,
> 
> Miguel Ferreira
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message