cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "SrikanteswaraRao Talluri" <srikanteswararao.tall...@citrix.com>
Subject Re: Review Request 19866: Added Fixes for CLOUDSTACK-6316
Date Tue, 01 Apr 2014 10:58:23 GMT

-----------------------------------------------------------
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
> 
>


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