You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Ashutosh Kelkar <as...@clogeny.com> on 2013/08/27 12:23:52 UTC

Review Request 13856: CLOUDSTACK-4303: Add missing base.py changes

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

Review request for cloudstack, Girish Shilamkar, Jayapal Reddy, and Prasanna Santhanam.


Bugs: CLOUDSTACK-4303


Repository: cloudstack-git


Description
-------

egressdefaultpolicy parameter was not set in base.py. Therefore
egress test failed. Added this missing change.


Diffs
-----

  tools/marvin/marvin/integration/lib/base.py b5d086b 

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


Testing
-------


Thanks,

Ashutosh Kelkar


Re: Review Request 13856: CLOUDSTACK-4303: Add missing base.py changes

Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13856/#review25605
-----------------------------------------------------------


Commit a7833cdfb4b4a3dfcf9b185241056620651b6eb1 in branch refs/heads/master from Ashutosh Kelkar
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=a7833cd ]

CLOUDSTACK-4303: Add missing base.py changes

egressdefaultpolicy parameter was not set in base.py. Therefore
egress test failed. Added this missing change.

Signed-off-by: Jayapal <ja...@apache.org>


- ASF Subversion and Git Services


On Aug. 27, 2013, 10:23 a.m., Ashutosh Kelkar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13856/
> -----------------------------------------------------------
> 
> (Updated Aug. 27, 2013, 10:23 a.m.)
> 
> 
> Review request for cloudstack, Girish Shilamkar, Jayapal Reddy, and Prasanna Santhanam.
> 
> 
> Bugs: CLOUDSTACK-4303
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> egressdefaultpolicy parameter was not set in base.py. Therefore
> egress test failed. Added this missing change.
> 
> 
> Diffs
> -----
> 
>   tools/marvin/marvin/integration/lib/base.py b5d086b 
> 
> Diff: https://reviews.apache.org/r/13856/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ashutosh Kelkar
> 
>


Re: Review Request 13856: CLOUDSTACK-4303: Add missing base.py changes

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

Ship it!


Ship It!

- Jayapal Reddy


On Aug. 27, 2013, 10:23 a.m., Ashutosh Kelkar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13856/
> -----------------------------------------------------------
> 
> (Updated Aug. 27, 2013, 10:23 a.m.)
> 
> 
> Review request for cloudstack, Girish Shilamkar, Jayapal Reddy, and Prasanna Santhanam.
> 
> 
> Bugs: CLOUDSTACK-4303
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> egressdefaultpolicy parameter was not set in base.py. Therefore
> egress test failed. Added this missing change.
> 
> 
> Diffs
> -----
> 
>   tools/marvin/marvin/integration/lib/base.py b5d086b 
> 
> Diff: https://reviews.apache.org/r/13856/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ashutosh Kelkar
> 
>


Re: Review Request 13856: CLOUDSTACK-4303: Add missing base.py changes

Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13856/#review25607
-----------------------------------------------------------


Commit 6e64fa53c62d927e6dc4c0053abc5d3f9f8329dc in branch refs/heads/4.2-forward from Ashutosh Kelkar
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=6e64fa5 ]

CLOUDSTACK-4303: Add missing base.py changes

egressdefaultpolicy parameter was not set in base.py. Therefore
egress test failed. Added this missing change.

Signed-off-by: Jayapal <ja...@apache.org>


- ASF Subversion and Git Services


On Aug. 27, 2013, 10:23 a.m., Ashutosh Kelkar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13856/
> -----------------------------------------------------------
> 
> (Updated Aug. 27, 2013, 10:23 a.m.)
> 
> 
> Review request for cloudstack, Girish Shilamkar, Jayapal Reddy, and Prasanna Santhanam.
> 
> 
> Bugs: CLOUDSTACK-4303
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> egressdefaultpolicy parameter was not set in base.py. Therefore
> egress test failed. Added this missing change.
> 
> 
> Diffs
> -----
> 
>   tools/marvin/marvin/integration/lib/base.py b5d086b 
> 
> Diff: https://reviews.apache.org/r/13856/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ashutosh Kelkar
> 
>