----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18971/#review38083 -----------------------------------------------------------
test/integration/smoke/test_secondary_storage.py <https://reviews.apache.org/r/18971/#comment70048> We are verifying only one element in list using for loop and skipping, other elements need to be verified? test/integration/smoke/test_secondary_storage.py <https://reviews.apache.org/r/18971/#comment70049> I believe this change is not required, it does not have much impact. We are not looking for config existence at other places as well. Also, if config does not match for other parameters away from zone, all those checks are still not existing. These checks are not required in all tests. - Santhosh Edukulla 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 > >
