cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Santhosh Edukulla" <santhosh.eduku...@citrix.com>
Subject Re: Review Request 22197: CLOUDSTACK-6776: Removed hard coded vlan ids from BVT test_non_contiguous_vlan.py
Date Tue, 03 Jun 2014 12:27:24 GMT

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



test/integration/smoke/test_non_contigiousvlan.py
<https://reviews.apache.org/r/22197/#comment79036>

    It seems we are not reading test data. Can we check that?



test/integration/smoke/test_non_contigiousvlan.py
<https://reviews.apache.org/r/22197/#comment79037>

    As well, can we run these changes once and see no issues if its possible?


- Santhosh Edukulla


On June 3, 2014, 11:45 a.m., Ashutosh Kelkar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/22197/
> -----------------------------------------------------------
> 
> (Updated June 3, 2014, 11:45 a.m.)
> 
> 
> Review request for cloudstack, Girish Shilamkar and Santhosh Edukulla.
> 
> 
> Bugs: CLOUDSTACK-6776
>     https://issues.apache.org/jira/browse/CLOUDSTACK-6776
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
>  The test case fails whenever hard coded values overlap with already configured VLAN
in the physical network. Removed hard code VLAN ids and used the common function to get unused
VLAN ids from the physical network dynamically.
> 
> 
> Diffs
> -----
> 
>   test/integration/smoke/test_non_contigiousvlan.py 90235ec 
> 
> Diff: https://reviews.apache.org/r/22197/diff/
> 
> 
> Testing
> -------
> 
> Yes.
> 
> Log:
> Test to update a physical network and extend its vlan ... === TestName: test_extendPhysicalNetworkVlan
| Status : SUCCESS ===
> ok
> 
> ----------------------------------------------------------------------
> Ran 1 test in 15.328s
> 
> OK
> ~
> ~
> 
> 
> Thanks,
> 
> Ashutosh Kelkar
> 
>


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