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/20 22:49:08 UTC
Review Request 28300: CLOUDSTACK-7956 : Fixed the script
'test_project_usage.py' - Register Template in the Project to test the
Template limits on the project
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28300/
-----------------------------------------------------------
Review request for cloudstack and sangeetha hariharan.
Bugs: CLOUDSTACK-7956
https://issues.apache.org/jira/browse/CLOUDSTACK-7956
Repository: cloudstack-git
Description
-------
Based on Fix for CLOUDSTACK-7394, Caller should be owner after creating template from snapshot/volume. This requires the test case test_01_template_usage to be changed. The test case should test the limits by registering the template in the project instead of creating templates from Volumes.
Diffs
-----
test/integration/component/test_project_usage.py 2627504
Diff: https://reviews.apache.org/r/28300/diff/
Testing
-------
Test Upload/ delete a template and verify correct usage is generated ... === TestName: test_01_template_usage | Status : SUCCESS ===
ok
----------------------------------------------------------------------
Ran 1 test in 312.740s
OK
Thanks,
Chandan Purushothama
Re: Review Request 28300: CLOUDSTACK-7956 : Fixed the script
'test_project_usage.py' - Register Template in the Project to test the
Template limits on the project
Posted by sangeetha hariharan <sa...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28300/#review62649
-----------------------------------------------------------
Ship it!
Ship It!
- sangeetha hariharan
On Nov. 20, 2014, 9:49 p.m., Chandan Purushothama wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/28300/
> -----------------------------------------------------------
>
> (Updated Nov. 20, 2014, 9:49 p.m.)
>
>
> Review request for cloudstack and sangeetha hariharan.
>
>
> Bugs: CLOUDSTACK-7956
> https://issues.apache.org/jira/browse/CLOUDSTACK-7956
>
>
> Repository: cloudstack-git
>
>
> Description
> -------
>
> Based on Fix for CLOUDSTACK-7394, Caller should be owner after creating template from snapshot/volume. This requires the test case test_01_template_usage to be changed. The test case should test the limits by registering the template in the project instead of creating templates from Volumes.
>
>
> Diffs
> -----
>
> test/integration/component/test_project_usage.py 2627504
>
> Diff: https://reviews.apache.org/r/28300/diff/
>
>
> Testing
> -------
>
> Test Upload/ delete a template and verify correct usage is generated ... === TestName: test_01_template_usage | Status : SUCCESS ===
> ok
>
> ----------------------------------------------------------------------
> Ran 1 test in 312.740s
>
> OK
>
>
> Thanks,
>
> Chandan Purushothama
>
>
Re: Review Request 28300: CLOUDSTACK-7956 : Fixed the script
'test_project_usage.py' - Register Template in the Project to test the
Template limits on the project
Posted by Sebastien Goasguen <ru...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28300/#review79015
-----------------------------------------------------------
Thank you for submitting your CloudStack contribution through review board. After discussion on the dev@cloudstack.apache.org the community decided to close down review board and start accepting contributiong through GitHub pull requests. We have been using GH PR for several months now and the process is better than review board.
We will keep Review Board open for another week to give you time to migrate your patch to a github PR if you wish. After that time, your patch will no longer be viewable (even though it will not be deleted).
Please consider submitting a pull request.
Great instructions are available at:
https://github.com/apache/cloudstack/blob/master/CONTRIBUTING.md
Thank you very much for your time and your contribution to Apache CloudStack, we hope that using this new process will encourage you to do more.
- Sebastien Goasguen
On Nov. 20, 2014, 9:49 p.m., Chandan Purushothama wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/28300/
> -----------------------------------------------------------
>
> (Updated Nov. 20, 2014, 9:49 p.m.)
>
>
> Review request for cloudstack and sangeetha hariharan.
>
>
> Bugs: CLOUDSTACK-7956
> https://issues.apache.org/jira/browse/CLOUDSTACK-7956
>
>
> Repository: cloudstack-git
>
>
> Description
> -------
>
> Based on Fix for CLOUDSTACK-7394, Caller should be owner after creating template from snapshot/volume. This requires the test case test_01_template_usage to be changed. The test case should test the limits by registering the template in the project instead of creating templates from Volumes.
>
>
> Diffs
> -----
>
> test/integration/component/test_project_usage.py 2627504
>
> Diff: https://reviews.apache.org/r/28300/diff/
>
>
> Testing
> -------
>
> Test Upload/ delete a template and verify correct usage is generated ... === TestName: test_01_template_usage | Status : SUCCESS ===
> ok
>
> ----------------------------------------------------------------------
> Ran 1 test in 312.740s
>
> OK
>
>
> Thanks,
>
> Chandan Purushothama
>
>