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/12/05 07:57:25 UTC

Review Request 16036: CLOUDSTACK-5377: Fixed test case dependency issue in snapshots test cases

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

Review request for cloudstack, Girish Shilamkar and SrikanteswaraRao Talluri.


Bugs: CLOUDSTACK-5377
    https://issues.apache.org/jira/browse/CLOUDSTACK-5377


Repository: cloudstack-git


Description
-------

component.test_snapshots.TestSnapshots.test_04_delete_snapshot failed with "Check list response returns a valid list".

The test case is failing while listing the data disks attached to virtual machine, but the data disk was being detached from the VM in test_07.

Removed this dependency between the test cases.


Diffs
-----

  test/integration/component/test_snapshots.py 0a82fc1 

Diff: https://reviews.apache.org/r/16036/diff/


Testing
-------

Ran the test case along with the test case on which it was previously dependent.

Log:
test_01_createVM_snapshotTemplate (test_snapshots.TestCreateVMSnapshotTemplate)
Test create VM, Snapshot and Template ... skipped 'skip'
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 ... ok
test_07_template_from_snapshot (test_snapshots.TestSnapshots)
Create Template from snapshot ... skipped 'skip'

----------------------------------------------------------------------
Ran 7 tests in 216.329s

OK (skipped=5)


Thanks,

Gaurav Aradhye


Re: Review Request 16036: CLOUDSTACK-5377: Fixed test case dependency issue in 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/16036/#review29803
-----------------------------------------------------------


Commit 25dd1156736dbbe9af4069b25b6b0457e0f55429 in branch refs/heads/4.2 from Gaurav Aradhye
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=25dd115 ]

CLOUDSTACK-5377: Fixed test case dependency issue in snapshots test cases


- ASF Subversion and Git Services


On Dec. 5, 2013, 6:57 a.m., Gaurav Aradhye wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16036/
> -----------------------------------------------------------
> 
> (Updated Dec. 5, 2013, 6:57 a.m.)
> 
> 
> Review request for cloudstack, Girish Shilamkar and SrikanteswaraRao Talluri.
> 
> 
> Bugs: CLOUDSTACK-5377
>     https://issues.apache.org/jira/browse/CLOUDSTACK-5377
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> component.test_snapshots.TestSnapshots.test_04_delete_snapshot failed with "Check list response returns a valid list".
> 
> The test case is failing while listing the data disks attached to virtual machine, but the data disk was being detached from the VM in test_07.
> 
> Removed this dependency between the test cases.
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_snapshots.py 0a82fc1 
> 
> Diff: https://reviews.apache.org/r/16036/diff/
> 
> 
> Testing
> -------
> 
> Ran the test case along with the test case on which it was previously dependent.
> 
> Log:
> test_01_createVM_snapshotTemplate (test_snapshots.TestCreateVMSnapshotTemplate)
> Test create VM, Snapshot and Template ... skipped 'skip'
> 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 ... ok
> test_07_template_from_snapshot (test_snapshots.TestSnapshots)
> Create Template from snapshot ... skipped 'skip'
> 
> ----------------------------------------------------------------------
> Ran 7 tests in 216.329s
> 
> OK (skipped=5)
> 
> 
> Thanks,
> 
> Gaurav Aradhye
> 
>


Re: Review Request 16036: CLOUDSTACK-5377: Fixed test case dependency issue in 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/16036/#review29801
-----------------------------------------------------------


Commit 60c180086e44fcb2fe700b997c83126d8004058e in branch refs/heads/4.3 from Gaurav Aradhye
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=60c1800 ]

CLOUDSTACK-5377: Fixed test case dependency issue in snapshots test cases


- ASF Subversion and Git Services


On Dec. 5, 2013, 6:57 a.m., Gaurav Aradhye wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16036/
> -----------------------------------------------------------
> 
> (Updated Dec. 5, 2013, 6:57 a.m.)
> 
> 
> Review request for cloudstack, Girish Shilamkar and SrikanteswaraRao Talluri.
> 
> 
> Bugs: CLOUDSTACK-5377
>     https://issues.apache.org/jira/browse/CLOUDSTACK-5377
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> component.test_snapshots.TestSnapshots.test_04_delete_snapshot failed with "Check list response returns a valid list".
> 
> The test case is failing while listing the data disks attached to virtual machine, but the data disk was being detached from the VM in test_07.
> 
> Removed this dependency between the test cases.
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_snapshots.py 0a82fc1 
> 
> Diff: https://reviews.apache.org/r/16036/diff/
> 
> 
> Testing
> -------
> 
> Ran the test case along with the test case on which it was previously dependent.
> 
> Log:
> test_01_createVM_snapshotTemplate (test_snapshots.TestCreateVMSnapshotTemplate)
> Test create VM, Snapshot and Template ... skipped 'skip'
> 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 ... ok
> test_07_template_from_snapshot (test_snapshots.TestSnapshots)
> Create Template from snapshot ... skipped 'skip'
> 
> ----------------------------------------------------------------------
> Ran 7 tests in 216.329s
> 
> OK (skipped=5)
> 
> 
> Thanks,
> 
> Gaurav Aradhye
> 
>


Re: Review Request 16036: CLOUDSTACK-5377: Fixed test case dependency issue in 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/16036/#review29800
-----------------------------------------------------------


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

CLOUDSTACK-5377: Fixed test case dependency issue in snapshots test cases


- ASF Subversion and Git Services


On Dec. 5, 2013, 6:57 a.m., Gaurav Aradhye wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16036/
> -----------------------------------------------------------
> 
> (Updated Dec. 5, 2013, 6:57 a.m.)
> 
> 
> Review request for cloudstack, Girish Shilamkar and SrikanteswaraRao Talluri.
> 
> 
> Bugs: CLOUDSTACK-5377
>     https://issues.apache.org/jira/browse/CLOUDSTACK-5377
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> component.test_snapshots.TestSnapshots.test_04_delete_snapshot failed with "Check list response returns a valid list".
> 
> The test case is failing while listing the data disks attached to virtual machine, but the data disk was being detached from the VM in test_07.
> 
> Removed this dependency between the test cases.
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_snapshots.py 0a82fc1 
> 
> Diff: https://reviews.apache.org/r/16036/diff/
> 
> 
> Testing
> -------
> 
> Ran the test case along with the test case on which it was previously dependent.
> 
> Log:
> test_01_createVM_snapshotTemplate (test_snapshots.TestCreateVMSnapshotTemplate)
> Test create VM, Snapshot and Template ... skipped 'skip'
> 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 ... ok
> test_07_template_from_snapshot (test_snapshots.TestSnapshots)
> Create Template from snapshot ... skipped 'skip'
> 
> ----------------------------------------------------------------------
> Ran 7 tests in 216.329s
> 
> OK (skipped=5)
> 
> 
> Thanks,
> 
> Gaurav Aradhye
> 
>


Re: Review Request 16036: CLOUDSTACK-5377: Fixed test case dependency issue in 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/16036/#review29802
-----------------------------------------------------------

Ship it!


Committed to 4.2, 4.3 and master.

- Girish Shilamkar


On Dec. 5, 2013, 6:57 a.m., Gaurav Aradhye wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16036/
> -----------------------------------------------------------
> 
> (Updated Dec. 5, 2013, 6:57 a.m.)
> 
> 
> Review request for cloudstack, Girish Shilamkar and SrikanteswaraRao Talluri.
> 
> 
> Bugs: CLOUDSTACK-5377
>     https://issues.apache.org/jira/browse/CLOUDSTACK-5377
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> component.test_snapshots.TestSnapshots.test_04_delete_snapshot failed with "Check list response returns a valid list".
> 
> The test case is failing while listing the data disks attached to virtual machine, but the data disk was being detached from the VM in test_07.
> 
> Removed this dependency between the test cases.
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_snapshots.py 0a82fc1 
> 
> Diff: https://reviews.apache.org/r/16036/diff/
> 
> 
> Testing
> -------
> 
> Ran the test case along with the test case on which it was previously dependent.
> 
> Log:
> test_01_createVM_snapshotTemplate (test_snapshots.TestCreateVMSnapshotTemplate)
> Test create VM, Snapshot and Template ... skipped 'skip'
> 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 ... ok
> test_07_template_from_snapshot (test_snapshots.TestSnapshots)
> Create Template from snapshot ... skipped 'skip'
> 
> ----------------------------------------------------------------------
> Ran 7 tests in 216.329s
> 
> OK (skipped=5)
> 
> 
> Thanks,
> 
> Gaurav Aradhye
> 
>