You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Chandan Purushothama <Ch...@citrix.com> on 2014/11/27 01:09:54 UTC

Review Request 28489: CLOUDSTACK-7978 : Fixed the script 'test_egress_rules.py' - Zone Network Type Information should to be passed to VirtualMachine create method

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

Review request for cloudstack and sangeetha hariharan.


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


Repository: cloudstack-git


Description
-------

Currently, test cases in the test suite do not pass the Zone Network Type to the Virtual Machine class create method. This doesn't cause any problem with majority of the configurations except EIP-ELB Configuration. Hence Zone Network Type Information should be passed consciously to the create method to cover EIP-ELB Configuration.


Diffs
-----

  test/integration/component/test_egress_rules.py 0f05c07 

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


Testing
-------

Executed One Test to prove the fix:

Test authorize ingress rule ... === TestName: test_authorizeIngressRule | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 1 test in 126.613s

OK


Thanks,

Chandan Purushothama


Re: Review Request 28489: CLOUDSTACK-7978 : Fixed the script 'test_egress_rules.py' - Zone Network Type Information should to be passed to VirtualMachine create method

Posted by sangeetha hariharan <sa...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28489/#review63164
-----------------------------------------------------------

Ship it!


Ship It!

- sangeetha hariharan


On Nov. 27, 2014, 12:09 a.m., Chandan Purushothama wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/28489/
> -----------------------------------------------------------
> 
> (Updated Nov. 27, 2014, 12:09 a.m.)
> 
> 
> Review request for cloudstack and sangeetha hariharan.
> 
> 
> Bugs: CLOUDSTACK-7978
>     https://issues.apache.org/jira/browse/CLOUDSTACK-7978
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Currently, test cases in the test suite do not pass the Zone Network Type to the Virtual Machine class create method. This doesn't cause any problem with majority of the configurations except EIP-ELB Configuration. Hence Zone Network Type Information should be passed consciously to the create method to cover EIP-ELB Configuration.
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_egress_rules.py 0f05c07 
> 
> Diff: https://reviews.apache.org/r/28489/diff/
> 
> 
> Testing
> -------
> 
> Executed One Test to prove the fix:
> 
> Test authorize ingress rule ... === TestName: test_authorizeIngressRule | Status : SUCCESS ===
> ok
> 
> ----------------------------------------------------------------------
> Ran 1 test in 126.613s
> 
> OK
> 
> 
> Thanks,
> 
> Chandan Purushothama
> 
>