----------------------------------------------------------- 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 > >
