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

Ship it!


Ship It!

- SrikanteswaraRao Talluri


On April 1, 2014, 10:04 a.m., Santhosh Edukulla wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19866/
> -----------------------------------------------------------
> 
> (Updated April 1, 2014, 10:04 a.m.)
> 
> 
> Review request for cloudstack and SrikanteswaraRao Talluri.
> 
> 
> Bugs: cloudstack-6316
>     https://issues.apache.org/jira/browse/cloudstack-6316
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> 1. Earlier, i added a facility called DeleteDataCenter to clean up a 
> datacenter, it has some issues for deleting, fixed them. This change is under 
> deployDataCenter.py
> 2. Earlier Added a new  facility, where we can deploy multiple zones and run 
> regression simultaneously on all zones with multiple hypervisors, 
> 2.1 So, user creates the data center with multiple zones and hypervisors.
> 2.2 Provides switches like zone and hypervisor to marvinPlugin and all tests 
> are run against that zone. With this patch, added hypervisor switch to 
> nose\marvinplugin.
> 3. Fixed an issue in one of the test suite under maint, it is using 
> wait_for_vm function, which is not available.
> 4. Added time taken for async job, post some stress scenarios, we want to 
> collect logs and analyze the time taken for various commands, their jobid and 
> time taken, added time taken
>  facility.
> 5. Added pep8 changes for marvin directory
> 
> 
> Diffs
> -----
> 
>   test/integration/component/maint/test_host_high_availability.py f81bff2 
>   tools/marvin/marvin/__init__.py 7af168e 
>   tools/marvin/marvin/asyncJobMgr.py ee3ae5a 
>   tools/marvin/marvin/cloudstackConnection.py 26116cc 
>   tools/marvin/marvin/cloudstackException.py 3da28b7 
>   tools/marvin/marvin/cloudstackTestCase.py 303f7ad 
>   tools/marvin/marvin/cloudstackTestClient.py 0dac5f4 
>   tools/marvin/marvin/codegenerator.py d4a81cf 
>   tools/marvin/marvin/config/test_data.py 563bc07 
>   tools/marvin/marvin/configGenerator.py 605832c 
>   tools/marvin/marvin/dbConnection.py ba7bac9 
>   tools/marvin/marvin/deployAndRun.py 56747a7 
>   tools/marvin/marvin/deployDataCenter.py d4a0b85 
>   tools/marvin/marvin/jsonHelper.py f7db5ba 
>   tools/marvin/marvin/marvinInit.py 7ceba78 
>   tools/marvin/marvin/marvinLog.py f62c35a 
>   tools/marvin/marvin/marvinPlugin.py 6525cd2 
>   tools/marvin/marvin/sandbox/advanced/advanced_env.py 1728e61 
>   tools/marvin/marvin/sandbox/advancedsg/advancedsg_env.py 9cf4a0a 
>   tools/marvin/marvin/sandbox/basic/basic_env.py bf106fc 
>   tools/marvin/marvin/sandbox/demo/simulator/simulator_setup.py d45d482 
>   tools/marvin/marvin/sshClient.py c24477c 
>   tools/marvin/marvin/tcExecuteEngine.py c437337 
>   tools/marvin/marvin/testSetupSuccess.py 1701626 
> 
> Diff: https://reviews.apache.org/r/19866/diff/
> 
> 
> Testing
> -------
> 
> Tested running creating and deleting deployDataCenter.
> 
> 
> Thanks,
> 
> Santhosh Edukulla
> 
>

Reply via email to