> On Nov. 10, 2014, 12:05 p.m., SrikanteswaraRao Talluri wrote: > > test/integration/testpaths/testpath_system_vm_ha.py, line 462 > > <https://reviews.apache.org/r/27321/diff/2/?file=747212#file747212line462> > > > > I feel this step is not related to this test.
We are just making sure none of the VM or System VM is still present on the host. If any was present, then it's migrated. I have updated the comment. > On Nov. 10, 2014, 12:05 p.m., SrikanteswaraRao Talluri wrote: > > test/integration/testpaths/testpath_system_vm_ha.py, line 532 > > <https://reviews.apache.org/r/27321/diff/2/?file=747212#file747212line532> > > > > how do you make sure that guest VMs and router VMs are on the same > > host. otherwise, you will not see them migrate to another host. Yes, we are only checking that no vm/system Vm is present on the host and if any it is moved to other host. - Ashutosh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27321/#review60579 ----------------------------------------------------------- On Nov. 3, 2014, 8:51 a.m., Ashutosh Kelkar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27321/ > ----------------------------------------------------------- > > (Updated Nov. 3, 2014, 8:51 a.m.) > > > Review request for cloudstack, suresh sadhu and SrikanteswaraRao Talluri. > > > Bugs: CLOUDSTACK-7815 > https://issues.apache.org/jira/browse/CLOUDSTACK-7815 > > > Repository: cloudstack-git > > > Description > ------- > > Convers around 12 scenarios. > More patches to follow if scenarios are automatable. > > > Diffs > ----- > > test/integration/testpaths/testpath_system_vm_ha.py PRE-CREATION > > Diff: https://reviews.apache.org/r/27321/diff/ > > > Testing > ------- > > Bring down the host on which SSVM is running ... === TestName: > test_01_bring_ssvm_host_down | Status : SUCCESS === > ok > Bring down the host on which CPVM is running ... === TestName: > test_02_bring_cpvm_host_down | Status : SUCCESS === > ok > Bring down the host on which router is running ... === TestName: > test_03_bring_router_host_down | Status : SUCCESS === > ok > > ---------------------------------------------------------------------- > Ran 3 tests in 1137.479s > > OK > > > Thanks, > > Ashutosh Kelkar > >
