----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25535/#review53322 -----------------------------------------------------------
Ship it! Ship It! - SrikanteswaraRao Talluri On Sept. 11, 2014, 10:32 a.m., Gaurav Aradhye wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25535/ > ----------------------------------------------------------- > > (Updated Sept. 11, 2014, 10:32 a.m.) > > > Review request for cloudstack, sanjeev n, Santhosh Edukulla, and > SrikanteswaraRao Talluri. > > > Bugs: CLOUDSTACK-7391 > https://issues.apache.org/jira/browse/CLOUDSTACK-7391 > > > Repository: cloudstack-git > > > Description > ------- > > virutalmachineid should be passed to listHosts api for retrieving the hosts > suitable for VM migration. > > Also fixed pep8 issues. > > > Diffs > ----- > > test/integration/component/maint/test_host_high_availability.py 4cd7fd8 > > Diff: https://reviews.apache.org/r/25535/diff/ > > > Testing > ------- > > Yes. > > Log: > Test VM deployments (Create HA enabled Compute Service Offering and VM) ... > SKIP: skip > Verify you can not create new VMs on hosts with an ha.tag ... SKIP: skip > Verify you can not migrate VMs to hosts with an ha.tag (positive) ... === > TestName: test_03_cant_migrate_vm_to_host_with_ha_positive | Status : SUCCESS > === > ok > Verify you can not migrate VMs to hosts with an ha.tag (negative) ... SKIP: > skip > Verify that none of the VMs with HA enabled migrate to an ha tagged host > during live migration ... SKIP: skip > Verify that none of the VMs without HA enabled migrate to an ha tagged host > during live migration ... SKIP: skip > > ---------------------------------------------------------------------- > Ran 6 tests in 191.673s > > OK (SKIP=5) > > > Thanks, > > Gaurav Aradhye > >
