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 2014/08/21 17:24:37 UTC

Review Request 24932: CLOUDSTACK-7386: Fixed cleanup issue in test_escalations_volumes.py

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

Review request for cloudstack and Santhosh Edukulla.


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


Repository: cloudstack-git


Description
-------

The test case failed while deleting snapshot in cleanup operation because the snapshot was already deleted in test case.
Removed the snapshot from cleanup list once it is deleted.


Diffs
-----

  test/integration/component/test_escalations_volumes.py d1dae12 

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


Testing
-------

Yes.

Log:
@summary: Test to verify pagination of snapshots for Volume ... === TestName: test_10_volume_snapshots_pagination | Status : SUCCESS ===
ok


Thanks,

Gaurav Aradhye


Re: Review Request 24932: CLOUDSTACK-7386: Fixed cleanup issue in test_escalations_volumes.py

Posted by Gaurav Aradhye <ga...@clogeny.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24932/
-----------------------------------------------------------

(Updated Aug. 22, 2014, 1:30 p.m.)


Review request for cloudstack, Girish Shilamkar and Santhosh Edukulla.


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


Repository: cloudstack-git


Description
-------

The test case failed while deleting snapshot in cleanup operation because the snapshot was already deleted in test case.
Removed the snapshot from cleanup list once it is deleted.


Diffs
-----

  test/integration/component/test_escalations_volumes.py d1dae12 

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


Testing
-------

Yes.

Log:
@summary: Test to verify pagination of snapshots for Volume ... === TestName: test_10_volume_snapshots_pagination | Status : SUCCESS ===
ok


Thanks,

Gaurav Aradhye