----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17113/#review32283 -----------------------------------------------------------
Ship it! Moving forward, could you please organise commits/patches in discrete functional units. So that it is easier to review the change. Thanks ! - Girish Shilamkar On Jan. 20, 2014, 6:36 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, 6:36 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 > >
