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

(Updated Jan. 22, 2014, 10:54 a.m.)


Review request for cloudstack and Girish Shilamkar.


Changes
-------

Attaching the new patch file as discussed. Tested to apply locally and it 
worked.


Repository: cloudstack-git


Description
-------

Fixed few issues including pep8 in previous submitted patch ( 17113 ). Tested 
Deploying a DC. 


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/17116/diff/


Testing
-------


File Attachments (updated)
----------------

New Patch File
  
https://reviews.apache.org/media/uploaded/files/2014/01/22/1759bdd3-40f9-4462-81b5-e44df20774dc__new_diff.patch


Thanks,

Santhosh Edukulla

Reply via email to