----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9752/#review17407 -----------------------------------------------------------
First - thanks for working on tests!! It's something we need lots of. Both of these files are missing the ASF license header, please add that in. test/integration/component/test_concurrent_snapshot.py <https://reviews.apache.org/r/9752/#comment36930> This doesn't look like valid python to me. Are you sure this is the intended content? - David Nalley On March 5, 2013, 5:15 p.m., sanjeev n wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9752/ > ----------------------------------------------------------- > > (Updated March 5, 2013, 5:15 p.m.) > > > Review request for cloudstack and Prasanna Santhanam. > > > Description > ------- > > Added two test cases for snapshots improvement feature. > > > Diffs > ----- > > test/integration/component/test_concurrent_snapshot.py PRE-CREATION > test/integration/component/test_snapshots_improvements.py PRE-CREATION > > Diff: https://reviews.apache.org/r/9752/diff/ > > > Testing > ------- > > Executed the tests in advanced zone setup. > > > Thanks, > > sanjeev n > >