----------------------------------------------------------- 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. Changes ------- Review changes. 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 (updated) ----- 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