Re: Review Request 28695: CLOUDSTACK-8020: test_snapshots.py - Fixed cleanup issue

2014-12-04 Thread SrikanteswaraRao Talluri

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

Ship it!


0a757b0f10ddf5f226b8b47ba3b172ed81277153 master
c2c7ee25edc1452fa247247f95bd8836706d9c46 4.5

- SrikanteswaraRao Talluri


On Dec. 4, 2014, 7:20 a.m., Ashutosh Kelkar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/28695/
 ---
 
 (Updated Dec. 4, 2014, 7:20 a.m.)
 
 
 Review request for cloudstack and SrikanteswaraRao Talluri.
 
 
 Bugs: CLOUDSTACK-8020
 https://issues.apache.org/jira/browse/CLOUDSTACK-8020
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 Fixed cleanup issue. Test case failed while deleting snapshot in the cleanup 
 opearation. Snapshot gets cleaned as a part of account cleanup. No need to 
 add it seperately to cleanup list.
 
 
 Diffs
 -
 
   test/integration/component/test_snapshots.py 62f2f06 
 
 Diff: https://reviews.apache.org/r/28695/diff/
 
 
 Testing
 ---
 
 Test Creating snapshot from volume having spaces in name(KVM) ... SKIP: skip
 Test Snapshot Data Disk ... SKIP: skip
 Test snapshot from detached disk ... === TestName: 
 test_03_snapshot_detachedDisk | Status : SUCCESS ===
 ok
 Test Delete Snapshot ... SKIP: skip
 Create Template from snapshot ... SKIP: skip
 
 --
 Ran 5 tests in 193.566s
 
 OK (SKIP=4)
 
 
 Thanks,
 
 Ashutosh Kelkar
 




Review Request 28695: CLOUDSTACK-8020: test_snapshots.py - Fixed cleanup issue

2014-12-04 Thread Ashutosh Kelkar

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

Review request for cloudstack and SrikanteswaraRao Talluri.


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


Repository: cloudstack-git


Description
---

Fixed cleanup issue. Test case failed while deleting snapshot in the cleanup 
opearation. Snapshot gets cleaned as a part of account cleanup. No need to add 
it seperately to cleanup list.


Diffs
-

  test/integration/component/test_snapshots.py 62f2f06 

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


Testing
---

Test Creating snapshot from volume having spaces in name(KVM) ... SKIP: skip
Test Snapshot Data Disk ... SKIP: skip
Test snapshot from detached disk ... === TestName: 
test_03_snapshot_detachedDisk | Status : SUCCESS ===
ok
Test Delete Snapshot ... SKIP: skip
Create Template from snapshot ... SKIP: skip

--
Ran 5 tests in 193.566s

OK (SKIP=4)


Thanks,

Ashutosh Kelkar