cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ashutosh Kelkar" <ashut...@clogeny.com>
Subject Re: Review Request 14058: Including tests for VPC VM Lifecycle on Tagged hosts
Date Mon, 30 Sep 2013 02:35:06 GMT


> On Sept. 26, 2013, 1:30 a.m., Sheng Yang wrote:
> > Please test it before submit.
> 
> Ashutosh Kelkar wrote:
>     The tests have been tested and run correctly. There is a manual step needed in setup
for these tests - Adding host tags to the hosts which is why the tests are skipped when committing.
> 
> Sheng Yang wrote:
>     No, looks like the test case added storage tag instead of host tag. Don't know how
it could run.

The tests don't add the tags to the host themselves. The host tagging is done manually before
running the tests. The tests don't create any tags storage or host. Could you please point
me to where you think this is happening in the tests?


- Ashutosh


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14058/#review26402
-----------------------------------------------------------


On Sept. 18, 2013, 2:28 p.m., Ashutosh Kelkar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/14058/
> -----------------------------------------------------------
> 
> (Updated Sept. 18, 2013, 2:28 p.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
> -------
> 
> 
> Thanks,
> 
> Ashutosh Kelkar
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message