----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13644/#review25342 -----------------------------------------------------------
Commit 9ed5083f9bd9eb668ea152635c98e1598c38f7c2 in branch refs/heads/master from Prasanna Santhanam [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=9ed5083 ] CLOUDSTACK-4257: Additional asserts for storage migration test Test whether valid storage pools are listed and at least one pool is available for migration. Signed-off-by: Prasanna Santhanam <t...@apache.org> (cherry picked from commit a82f66e0fa8c9e20b0468aae82871c46a9f8b67e) - ASF Subversion and Git Services On Aug. 19, 2013, 11:10 a.m., Saksham Srivastava wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13644/ > ----------------------------------------------------------- > > (Updated Aug. 19, 2013, 11:10 a.m.) > > > Review request for cloudstack, Devdeep Singh and Prasanna Santhanam. > > > Bugs: CLOUDSTACK-4257 > > > Repository: cloudstack-git > > > Description > ------- > > Test case > integration.component.test_storage_motion.TestStorageMotion.test_02_migrate_volume > failed during api call liststorage pool with unexpected result > AssertionError: Check list storage pools response for valid list > > This happens when there are no suitable pools found for migration. > Added check for empty pool list and then skip the test if no suitable hosts > are found. > > > Diffs > ----- > > test/integration/component/test_storage_motion.py eda77d1 > > Diff: https://reviews.apache.org/r/13644/diff/ > > > Testing > ------- > > If there is a single pool in the cluster, the test is skipped. > If multiple suitable hosts are found, the test passes. > > > Thanks, > > Saksham Srivastava > >