----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13736/#review25466 -----------------------------------------------------------
Also please remove the 'print' statements as they won't show in the testrunner - Prasanna Santhanam On Aug. 23, 2013, 10:03 a.m., Gaurav Aradhye wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13736/ > ----------------------------------------------------------- > > (Updated Aug. 23, 2013, 10:03 a.m.) > > > Review request for cloudstack and Prasanna Santhanam. > > > Repository: cloudstack-git > > > Description > ------- > > Resolved cloudstack: 4452 > Changes in function is_snapshot_on_nfs to check if the snapshot is present in > the secondary storage. > > 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 snapshot. > > 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 > ----- > > test/integration/component/test_snapshot_gc.py 3cd4194 > test/integration/component/test_snapshot_limits.py 1bc2798 > test/integration/component/test_snapshots.py cc2e604 > tools/marvin/marvin/integration/lib/utils.py 7863bfb > > Diff: https://reviews.apache.org/r/13736/diff/ > > > Testing > ------- > > > Thanks, > > Gaurav Aradhye > >
