----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18971/#review37169 -----------------------------------------------------------
test/integration/smoke/test_secondary_storage.py <https://reviews.apache.org/r/18971/#comment68577> i would suggest that, if the zone information in the config doesn't match with actual zone, then we should still proceed by collecting the actual zone, cluster, pod details from the zone instead of skipping the test. Purpose of this test is to just list* APIs. - SrikanteswaraRao Talluri On March 10, 2014, 4:28 p.m., Gaurav Aradhye wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18971/ > ----------------------------------------------------------- > > (Updated March 10, 2014, 4:28 p.m.) > > > Review request for cloudstack, Santhosh Edukulla and SrikanteswaraRao Talluri. > > > Bugs: CLOUDSTACK-6216 > https://issues.apache.org/jira/browse/CLOUDSTACK-6216 > > > Repository: cloudstack-git > > > Description > ------- > > Test case was failing when it is run on an existing setup because the zones > would not match with those mentioned in the config file. > Skipped the test case in this case. > > > Diffs > ----- > > test/integration/smoke/test_secondary_storage.py ff9692f > > Diff: https://reviews.apache.org/r/18971/diff/ > > > Testing > ------- > > Yes. Ran on an existing setup. > Log: > Test system VM start ... SKIP: skip > Test system templates are ready ... SKIP: Zone in config file not present in > setup > > ---------------------------------------------------------------------- > Ran 2 tests in 0.012s > > OK (SKIP=2) > > > Thanks, > > Gaurav Aradhye > >
