You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Gaurav Aradhye <ga...@clogeny.com> on 2013/09/12 08:59:04 UTC

Re: Review Request 13559: Automation: Portable IP test cases

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

(Updated Sept. 12, 2013, 12:28 p.m.)


Review request for cloudstack, Murali Reddy, venkata swamy babu  budumuru, and Prasanna Santhanam.


Changes
-------

Review changes.

For running portable ip test cases, following config values should be filled up, else tests will skip.

"portableIpRange":
    {
        "gateway" : "x.x.x.x",
        "netmask" : "x.x.x.x",
        "startip" : "x.x.x.x",
        "endip"   : "x.x.x.x",
        "vlan"    : "x"
    }

vlan is not compulsary, if not stated, it will be untagged by default.


Repository: cloudstack-git


Description
-------

Adding new test cases for feature Portable IP


Diffs (updated)
-----

  test/integration/component/test_portable_ip.py PRE-CREATION 
  tools/marvin/marvin/integration/lib/base.py 3016ee4 
  tools/marvin/marvin/integration/lib/common.py 7e8d92d 

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


Testing
-------


Thanks,

Gaurav Aradhye


Re: Review Request 13559: Automation: Portable IP test cases

Posted by Gaurav Aradhye <ga...@clogeny.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13559/
-----------------------------------------------------------

(Updated Sept. 19, 2013, 12:57 p.m.)


Review request for cloudstack, Murali Reddy, venkata swamy babu  budumuru, and Prasanna Santhanam.


Changes
-------

Updated patch with missing method "get_region". This was missing in earlier patch.


Repository: cloudstack-git


Description
-------

Adding new test cases for feature Portable IP


Diffs (updated)
-----

  test/integration/component/test_portable_ip.py PRE-CREATION 
  tools/marvin/marvin/integration/lib/base.py fa4cc82 
  tools/marvin/marvin/integration/lib/common.py 6ffe951 

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


Testing
-------


Thanks,

Gaurav Aradhye


Re: Review Request 13559: Automation: Portable IP test cases

Posted by Gaurav Aradhye <ga...@clogeny.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13559/
-----------------------------------------------------------

(Updated Sept. 13, 2013, 4:52 p.m.)


Review request for cloudstack, Murali Reddy, venkata swamy babu  budumuru, and Prasanna Santhanam.


Changes
-------

Patch against 4.2-forward.


Repository: cloudstack-git


Description
-------

Adding new test cases for feature Portable IP


Diffs (updated)
-----

  test/integration/component/test_portable_ip.py PRE-CREATION 
  tools/marvin/marvin/integration/lib/base.py fa4cc82 
  tools/marvin/marvin/integration/lib/common.py 6ffe951 

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


Testing
-------


Thanks,

Gaurav Aradhye


Re: Review Request 13559: Automation: Portable IP test cases

Posted by Gaurav Aradhye <ga...@clogeny.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13559/
-----------------------------------------------------------

(Updated Sept. 13, 2013, 4:12 p.m.)


Review request for cloudstack, Murali Reddy, venkata swamy babu  budumuru, and Prasanna Santhanam.


Changes
-------

Last patch couldn't apply. Putting new patch as requested.


Repository: cloudstack-git


Description
-------

Adding new test cases for feature Portable IP


Diffs (updated)
-----

  test/integration/component/test_portable_ip.py PRE-CREATION 
  tools/marvin/marvin/integration/lib/base.py 3016ee4 
  tools/marvin/marvin/integration/lib/common.py 7e8d92d 

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


Testing
-------


Thanks,

Gaurav Aradhye


Re: Review Request 13559: Automation: Portable IP test cases

Posted by Gaurav Aradhye <ga...@clogeny.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13559/
-----------------------------------------------------------

(Updated Sept. 12, 2013, 5:44 p.m.)


Review request for cloudstack, Murali Reddy, venkata swamy babu  budumuru, and Prasanna Santhanam.


Changes
-------

Removed trailing white-spaces


Repository: cloudstack-git


Description
-------

Adding new test cases for feature Portable IP


Diffs (updated)
-----

  test/integration/component/test_portable_ip.py PRE-CREATION 
  tools/marvin/marvin/integration/lib/base.py 3016ee4 
  tools/marvin/marvin/integration/lib/common.py 7e8d92d 

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


Testing
-------


Thanks,

Gaurav Aradhye