You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Gaurav Aradhye <ga...@clogeny.com> on 2013/11/08 08:07:22 UTC

Review Request 15342: CLOUDSTACK-4648: Fixed Snapshots test cases

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15342/
-----------------------------------------------------------

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


Re: Review Request 15342: CLOUDSTACK-4648: Fixed Snapshots test cases

Posted by Girish Shilamkar <gi...@clogeny.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15342/#review28507
-----------------------------------------------------------

Ship it!


Ship It!

- Girish Shilamkar


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
> 
>


Re: Review Request 15342: CLOUDSTACK-4648: Fixed Snapshots test cases

Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15342/#review28509
-----------------------------------------------------------


Commit 3c350ab0b8ce1687a8f7e27e9e2f8e8d0fa68c47 in branch refs/heads/master from Gaurav Aradhye
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=3c350ab ]

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 <gi...@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
> 
>


Re: Review Request 15342: CLOUDSTACK-4648: Fixed Snapshots test cases

Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
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 <gi...@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
> 
>