You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Santhosh Edukulla <sa...@citrix.com> on 2014/05/21 12:05:49 UTC

Review Request 20623: Added version check, fixed data corruption, fixed regression issues

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

Review request for cloudstack, daan Hoogland and Jayapal Reddy.


Bugs: CLOUDSTACK-6736
    https://issues.apache.org/jira/browse/CLOUDSTACK-6736


Repository: cloudstack-git


Description
-------

1. ParsedConfig is getting corrupted when used in tests. Fixed now.
2. setup.py does not have minimal version requirements for various packages. Added that.
3. Fixed few misc script issues and removed print statements.
4. Added bug information to failed cases.


Diffs
-----

  test/integration/smoke/test_deploy_vm.py db26980 
  test/integration/smoke/test_guest_vlan_range.py bfef4a8 
  test/integration/smoke/test_hosts.py 7798c8e 
  test/integration/smoke/test_vm_life_cycle.py 85033c5 
  tools/marvin/marvin/cloudstackTestClient.py 86b0a4b 
  tools/marvin/setup.py 4c775ad 

Diff: https://reviews.apache.org/r/20623/diff/


Testing
-------

Ran changes against simulator


Thanks,

Santhosh Edukulla


Re: Review Request 20623: Added version check, fixed data corruption, fixed regression issues

Posted by Jayapal Reddy <ja...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20623/#review43606
-----------------------------------------------------------

Ship it!


Ship It!

- Jayapal Reddy


On May 21, 2014, 11:12 a.m., Santhosh Edukulla wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20623/
> -----------------------------------------------------------
> 
> (Updated May 21, 2014, 11:12 a.m.)
> 
> 
> Review request for cloudstack, daan Hoogland and Jayapal Reddy.
> 
> 
> Bugs: CLOUDSTACK-6736
>     https://issues.apache.org/jira/browse/CLOUDSTACK-6736
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> 1. ParsedConfig is getting corrupted when used in tests. Fixed now.
> 2. setup.py does not have minimal version requirements for various packages. Added that.
> 3. Fixed few misc script issues and removed print statements.
> 4. Added bug information to failed cases.
> 
> 
> Diffs
> -----
> 
>   test/integration/smoke/test_deploy_vm.py db26980 
>   test/integration/smoke/test_guest_vlan_range.py bfef4a8 
>   test/integration/smoke/test_hosts.py 7798c8e 
>   test/integration/smoke/test_vm_life_cycle.py 85033c5 
>   tools/marvin/marvin/cloudstackTestClient.py 86b0a4b 
>   tools/marvin/setup.py 4c775ad 
> 
> Diff: https://reviews.apache.org/r/20623/diff/
> 
> 
> Testing
> -------
> 
> Ran changes against simulator
> 
> 
> Thanks,
> 
> Santhosh Edukulla
> 
>


Re: Review Request 20623: Added version check, fixed data corruption, fixed regression issues

Posted by Santhosh Edukulla <sa...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20623/
-----------------------------------------------------------

(Updated May 21, 2014, 11:12 a.m.)


Review request for cloudstack, daan Hoogland and Jayapal Reddy.


Bugs: CLOUDSTACK-6736
    https://issues.apache.org/jira/browse/CLOUDSTACK-6736


Repository: cloudstack-git


Description
-------

1. ParsedConfig is getting corrupted when used in tests. Fixed now.
2. setup.py does not have minimal version requirements for various packages. Added that.
3. Fixed few misc script issues and removed print statements.
4. Added bug information to failed cases.


Diffs (updated)
-----

  test/integration/smoke/test_deploy_vm.py db26980 
  test/integration/smoke/test_guest_vlan_range.py bfef4a8 
  test/integration/smoke/test_hosts.py 7798c8e 
  test/integration/smoke/test_vm_life_cycle.py 85033c5 
  tools/marvin/marvin/cloudstackTestClient.py 86b0a4b 
  tools/marvin/setup.py 4c775ad 

Diff: https://reviews.apache.org/r/20623/diff/


Testing
-------

Ran changes against simulator


Thanks,

Santhosh Edukulla


Re: Review Request 20623: Added version check, fixed data corruption, fixed regression issues

Posted by Santhosh Edukulla <sa...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20623/
-----------------------------------------------------------

(Updated May 21, 2014, 10:55 a.m.)


Review request for cloudstack, daan Hoogland and Jayapal Reddy.


Changes
-------

Fixed spaces issue.


Bugs: CLOUDSTACK-6736
    https://issues.apache.org/jira/browse/CLOUDSTACK-6736


Repository: cloudstack-git


Description
-------

1. ParsedConfig is getting corrupted when used in tests. Fixed now.
2. setup.py does not have minimal version requirements for various packages. Added that.
3. Fixed few misc script issues and removed print statements.
4. Added bug information to failed cases.


Diffs (updated)
-----

  test/integration/smoke/test_deploy_vm.py db26980 
  test/integration/smoke/test_guest_vlan_range.py bfef4a8 
  test/integration/smoke/test_hosts.py 7798c8e 
  test/integration/smoke/test_vm_life_cycle.py 85033c5 
  tools/marvin/marvin/cloudstackTestClient.py 86b0a4b 
  tools/marvin/setup.py 4c775ad 

Diff: https://reviews.apache.org/r/20623/diff/


Testing
-------

Ran changes against simulator


Thanks,

Santhosh Edukulla