----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29245/#review65905 -----------------------------------------------------------
Ship it! fc2c1a09a7064aa9121ead7c9f88fb268698c22e master 125c1c27fc0ebe3781fd2fb63e2d2dd5689c9fa0 4.5 - SrikanteswaraRao Talluri On Dec. 19, 2014, 12:12 p.m., Gaurav Aradhye wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29245/ > ----------------------------------------------------------- > > (Updated Dec. 19, 2014, 12:12 p.m.) > > > Review request for cloudstack and SrikanteswaraRao Talluri. > > > Bugs: CLOUDSTACK-8098 > https://issues.apache.org/jira/browse/CLOUDSTACK-8098 > > > Repository: cloudstack-git > > > Description > ------- > > The test case fails while reverting the VM snapshot because the snapshot does > not contain the memory and VM is not in stopped state. > > Solution: > Stop the VM before reverting the snapshot > > Also fixed pep8 issues and imports. > > > Diffs > ----- > > test/integration/smoke/test_vm_snapshots.py 131da99 > > Diff: https://reviews.apache.org/r/29245/diff/ > > > Testing > ------- > > Yes. > > Log: > Test to create VM snapshots ... === TestName: test_01_create_vm_snapshots | > Status : SUCCESS === > ok > Test to revert VM snapshots ... === TestName: test_02_revert_vm_snapshots | > Status : SUCCESS === > ok > Test to delete vm snapshots ... === TestName: test_03_delete_vm_snapshots | > Status : SUCCESS === > ok > > ---------------------------------------------------------------------- > Ran 3 tests in 759.977s > > OK > > > Thanks, > > Gaurav Aradhye > >