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



test/integration/component/test_cpu_limits.py
<https://reviews.apache.org/r/10828/#comment40924>

    ASF license header is missing.



test/integration/component/test_cpu_limits.py
<https://reviews.apache.org/r/10828/#comment40925>

    detect from cls.get_zone().networktype? This should prevent it from being 
reqd in the settings.



test/integration/component/test_cpu_limits.py
<https://reviews.apache.org/r/10828/#comment40926>

    Is the entire suite skipped? Can the tests in this suite be run as they are?


- Prasanna Santhanam


On April 29, 2013, 2:29 a.m., Chirag Jog wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10828/
> -----------------------------------------------------------
> 
> (Updated April 29, 2013, 2:29 a.m.)
> 
> 
> Review request for cloudstack, Prasanna Santhanam and Girish Shilamkar.
> 
> 
> Description
> -------
> 
> Here is the detailed test plan: 
> https://issues.apache.org/jira/secure/attachment/12571551/LimitResourcesTestPlan1.xlsx
> 
> 
> This addresses bug https://issues.apache.org/jira/browse/CLOUDSTACK-1466.
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_cpu_limits.py PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/10828/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Chirag Jog
> 
>

Reply via email to