Review Request 13736: Resolved Cloudstack: 4452 - fixed test cases in test_snapshots.py

2013-08-22 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13736/ --- Review request for cloudstack and Prasanna Santhanam. Repository: cloudstack-gi

Re: Review Request 13736: Resolved Cloudstack: 4452 - fixed test cases in test_snapshots.py

2013-08-22 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13736/#review25459 --- Gaurav, there's more occurrences of is_snapshot_on_nfs in the same s

Re: Review Request 13736: Resolved Cloudstack: 4452 - fixed test cases in test_snapshots.py

2013-08-23 Thread Gaurav Aradhye
--- 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

Re: Review Request 13736: Resolved Cloudstack: 4452 - fixed test cases in test_snapshots.py

2013-08-23 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13736/#review25465 --- dir_paths should not be a required argument. You can simply mount th

Re: Review Request 13736: Resolved Cloudstack: 4452 - fixed test cases in test_snapshots.py

2013-08-23 Thread Prasanna Santhanam
--- 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

Re: Review Request 13736: Resolved Cloudstack: 4452 - fixed test cases in test_snapshots.py

2013-08-23 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13736/ --- (Updated Aug. 23, 2013, 11:52 a.m.) Review request for cloudstack and Prasanna

Re: Review Request 13736: Resolved Cloudstack: 4452 - fixed test cases in test_snapshots.py

2013-08-26 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13736/#review25538 --- Ship it! Applied to master and 4.2-forward but with additional chan