cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gaurav Aradhye" <>
Subject Re: Review Request 13736: Resolved Cloudstack: 4452 - fixed test cases in
Date Fri, 23 Aug 2013 11:52:58 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Aug. 23, 2013, 11:52 a.m.)

Review request for cloudstack and Prasanna Santhanam.


Removed print statements.
This also includes the fix for 4472. Will discard separate patch added for 4472.

Repository: cloudstack-git


Resolved cloudstack: 4452
Changes in function is_snapshot_on_nfs to check if the snapshot is present in the secondary

Earlier it was trying to match the name of the snapshot present on the storage with the UUID
of the snapshot which is incorrect way.
There's apparently no relation between the snapshot name on the storage and the UUID of the

Changed the function to check if the snapshotPath is valid and is indeed a "File".

Also removed the VM creation (virtual_machine_without_disk) step in setupClass of class TestSnapshots.
It is not used anywhere and was eating up the running time of test case and resources.

Diffs (updated)

  test/integration/component/ 3cd4194 
  test/integration/component/ 1bc2798 
  test/integration/component/ cc2e604 
  tools/marvin/marvin/integration/lib/ 7863bfb 




Gaurav Aradhye

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message