----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15342/#review28508 -----------------------------------------------------------
Commit b9547b07c059d229e6aed6213366cc3ab59fa26e in branch refs/heads/4.2 from Gaurav Aradhye [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=b9547b0 ] CLOUDSTACK-4648: Fixed Snapshots test cases Fixed following test cases: 1) test_01_volume_from_snapshot 2) test_03_snapshot_detachedDisk 3) test_07_template_from_snapshot All three test cases are running successfully on XenServer but test cases (01 and 07) failing on KVM due to a common issue. Product defect logged for this: https://issues.apache.org/jira/browse/CLOUDSTACK-5097 test_03 is running ok on KVM too, the problem was - disk was not getting detached because it was not unmounted before detaching. Added code to unmount the disk. Signed-off-by: Girish Shilamkar <gir...@clogeny.com> - ASF Subversion and Git Services On Nov. 8, 2013, 7:07 a.m., Gaurav Aradhye wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15342/ > ----------------------------------------------------------- > > (Updated Nov. 8, 2013, 7:07 a.m.) > > > Review request for cloudstack, Girish Shilamkar and SrikanteswaraRao Talluri. > > > Bugs: CLOUDSTACK-4648 > https://issues.apache.org/jira/browse/CLOUDSTACK-4648 > > > Repository: cloudstack-git > > > Description > ------- > > Fixed following test cases: > 1) test_01_volume_from_snapshot > 2) test_03_snapshot_detachedDisk > 3) test_07_template_from_snapshot > > All three test cases are running successfully on XenServer but test cases (01 > and 07) failing on KVM due to a common issue. > Product defect logged for this: > https://issues.apache.org/jira/browse/CLOUDSTACK-5097 > > test_03 is running ok on KVM too, the problem was - disk was not getting > detached because it was not unmounted before detaching. Added code to unmount > the disk. > > > Diffs > ----- > > test/integration/component/test_snapshots.py 1c2537c > tools/marvin/marvin/integration/lib/common.py 164ef20 > > Diff: https://reviews.apache.org/r/15342/diff/ > > > Testing > ------- > > Log on Xen: > > test_01_createVM_snapshotTemplate > (test_snapshots.TestCreateVMSnapshotTemplate) > Test create VM, Snapshot and Template ... ok > test_05_snapshot_events (test_snapshots.TestSnapshotEvents) > Test snapshot events ... skipped 'Skip' > test_01_volume_from_snapshot (test_snapshots.TestSnapshots) > Test Creating snapshot from volume having spaces in name(KVM) ... skipped > 'Skip' > test_02_snapshot_data_disk (test_snapshots.TestSnapshots) > Test Snapshot Data Disk ... skipped 'Skip' > test_03_snapshot_detachedDisk (test_snapshots.TestSnapshots) > Test snapshot from detached disk ... ok > test_04_delete_snapshot (test_snapshots.TestSnapshots) > Test Delete Snapshot ... skipped 'Skip' > test_07_template_from_snapshot (test_snapshots.TestSnapshots) > Create Template from snapshot ... ok > > ---------------------------------------------------------------------- > Ran 7 tests in 529.398s > > OK (skipped=4) > > > Thanks, > > Gaurav Aradhye > >