> On Oct. 16, 2013, 1:53 p.m., suresh sadhu wrote: > > reviewed the code and added host tags in the code as per the review > > comments but when we ran the TestVMLifeCycleDiffHosts tests,it failed with > > Host.Update method is undefined in our environment.can you please rerun and > > confirm once. > > > >
Hi Suresh, Looks like the new method got missed in the patch. I am updating the patch with it. - Ashutosh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14058/#review27064 ----------------------------------------------------------- On Oct. 9, 2013, 11:28 a.m., Ashutosh Kelkar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/14058/ > ----------------------------------------------------------- > > (Updated Oct. 9, 2013, 11:28 a.m.) > > > Review request for cloudstack, Girish Shilamkar, venkata swamy babu > budumuru, and Sheng Yang. > > > Repository: cloudstack-git > > > Description > ------- > > Added 10 tests for VPC VM liftcycle on tagged hosts > > New class added : TestVMLifeCycleDiffHosts > > def test_01_deploy_instance_in_network(self): > def test_02_stop_instance_in_network(self): > def test_03_start_instance_in_network(self): > def test_04_reboot_instance_in_network(self): > def test_05_destroy_instance_in_network(self): > def test_06_recover_instance_in_network(self): > def test_07_migrate_instance_in_network(self): > def test_08_user_data(self): > def test_09_meta_data(self): > def test_10_expunge_instance_in_network(self): > > This set of tests requires a multi host tagged setup (3 hosts - 2 hosts with > tag 'host1' and 1 host with tag 'host2') > > > Diffs > ----- > > test/integration/component/test_vpc_vm_life_cycle.py 9844c1f > > Diff: https://reviews.apache.org/r/14058/diff/ > > > Testing > ------- > > Tested locally on KVM advanced setup. > > Log: > > test_01_deploy_instance_in_network > (test_vpc_vm_life_cycle.TestVMLifeCycleDiffHosts) > Test deploy an instance in VPC networks ... ok > test_02_stop_instance_in_network > (test_vpc_vm_life_cycle.TestVMLifeCycleDiffHosts) > Test stop an instance in VPC networks ... ok > test_03_start_instance_in_network > (test_vpc_vm_life_cycle.TestVMLifeCycleDiffHosts) > Test start an instance in VPC networks ... ok > test_04_reboot_instance_in_network > (test_vpc_vm_life_cycle.TestVMLifeCycleDiffHosts) > Test reboot an instance in VPC networks ... ok > test_05_destroy_instance_in_network > (test_vpc_vm_life_cycle.TestVMLifeCycleDiffHosts) > Test destroy an instance in VPC networks ... ok > test_06_migrate_instance_in_network > (test_vpc_vm_life_cycle.TestVMLifeCycleDiffHosts) > Test migrate an instance in VPC networks ... ok > test_07_user_data (test_vpc_vm_life_cycle.TestVMLifeCycleDiffHosts) > Test user data in virtual machines ... ok > test_08_meta_data (test_vpc_vm_life_cycle.TestVMLifeCycleDiffHosts) > Test meta data in virtual machines ... ok > test_09_expunge_instance_in_network > (test_vpc_vm_life_cycle.TestVMLifeCycleDiffHosts) > Test expunge an instance in VPC networks ... ok > > > Thanks, > > Ashutosh Kelkar > >