cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Girish Shilamkar" <gir...@clogeny.com>
Subject Re: Review Request 17113: Phase1 Marvin Changes.Fix-for-CLOUDSTACK-5674, 5498, 5500
Date Mon, 20 Jan 2014 09:00:42 GMT

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


Santhosh,

I have already committed the orignal patch. Can you please post a patch on top of the old
one ?

Regards,
Girish

- Girish Shilamkar


On Jan. 20, 2014, 8:42 a.m., Santhosh Edukulla wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/17113/
> -----------------------------------------------------------
> 
> (Updated Jan. 20, 2014, 8:42 a.m.)
> 
> 
> Review request for cloudstack and Girish Shilamkar.
> 
> 
> Bugs: CLOUDSTACK-5498, CLOUDSTACK-5500 and CLOUDSTACK-5674
>     https://issues.apache.org/jira/browse/CLOUDSTACK-5498
>     https://issues.apache.org/jira/browse/CLOUDSTACK-5500
>     https://issues.apache.org/jira/browse/CLOUDSTACK-5674
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> 1. Added-Fix-for-CLOUDSTACK-5674,5498,5500-and other changes mentioned @ the link below:

> https://cwiki.apache.org/confluence/display/CLOUDSTACK/Phase1%3A+Marvin+Changes
> 
> 2. The changes and the patch need to be applied to separate topic branch created for
Marvin.
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/marvin
> 
> 3. Few more changes pending for tests suites need to be done and will  call for a merge
to main
> branches post that. 
> 
> Girish,
> 
> As discussed, submitted the patch. This patch needs to be applied "only" to new topic
branch created.
> 
> 
> Diffs
> -----
> 
>   setup/dev/advanced.cfg 23981f0 
>   setup/dev/basic.cfg 3e39d6d 
>   test/integration/component/test_add_remove_network.py 8ec2971 
>   test/integration/component/test_affinity_groups.py 7e4fabe 
>   test/integration/component/test_cpu_domain_limits.py c427e4f 
>   test/integration/component/test_cpu_limits.py bdf2869 
>   test/integration/component/test_cpu_max_limits.py 317df16 
>   test/integration/component/test_cpu_project_limits.py a8a1b3c 
>   test/integration/component/test_memory_limits.py 7921e4b 
>   test/integration/component/test_mm_domain_limits.py 68660c1 
>   test/integration/component/test_mm_max_limits.py e10c119 
>   test/integration/component/test_mm_project_limits.py c314011 
>   test/integration/component/test_portable_ip.py b09daf9 
>   test/integration/component/test_vpc.py 1af8d81 
>   test/integration/component/test_vpn_users.py 02dd026 
>   test/integration/smoke/test_affinity_groups.py c96a580 
>   test/integration/smoke/test_deploy_vm.py 425aeb7 
>   test/integration/smoke/test_deploy_vm_with_userdata.py e3788cf 
>   tools/marvin/marvin/asyncJobMgr.py ee0e891 
>   tools/marvin/marvin/cloudstackConnection.py fb03e3b 
>   tools/marvin/marvin/cloudstackException.py 6200003 
>   tools/marvin/marvin/cloudstackTestClient.py 4ac510b 
>   tools/marvin/marvin/codegenerator.py e0f056f 
>   tools/marvin/marvin/codes.py 74fb05d 
>   tools/marvin/marvin/config/test_data.cfg PRE-CREATION 
>   tools/marvin/marvin/configGenerator.py 0d79e8e 
>   tools/marvin/marvin/dbConnection.py 99014ab 
>   tools/marvin/marvin/deployDataCenter.py c4f6e1e 
>   tools/marvin/marvin/integration/lib/common.py 550de1a 
>   tools/marvin/marvin/jsonHelper.py ae40b8d 
>   tools/marvin/marvin/marvinInit.py f722058 
>   tools/marvin/marvin/marvinLog.py 76de185 
>   tools/marvin/marvin/marvinPlugin.py df7d7a3 
>   tools/marvin/marvin/tcExecuteEngine.py f959e7e 
> 
> Diff: https://reviews.apache.org/r/17113/diff/
> 
> 
> Testing
> -------
> 
> Few more changes pending for tests suites need to be done and will  call for a merge
to main
> branches post that. 
> 
> 
> Thanks,
> 
> Santhosh Edukulla
> 
>


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