You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Sowmya Krishnan <so...@citrix.com> on 2013/07/23 18:01:56 UTC

Review Request 12864: CLOUDSTACK-3697 Fix cleanup LB failure

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

Review request for cloudstack and Prasanna Santhanam.


Bugs: CLOUDSTACK-3697


Repository: cloudstack-git


Description
-------

LB is already deleted when project is cleaned up. Removed it from cleanup queue


Diffs
-----

  test/integration/component/test_project_resources.py 378238e 

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


Testing
-------

Tested script with fix locally and it passes now.


Thanks,

Sowmya Krishnan


Re: Review Request 12864: CLOUDSTACK-3697 Fix cleanup LB failure

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/12864/#review23718
-----------------------------------------------------------


Commit cfffe63afb60be77d43448ddcad18baf5f76d781 in branch refs/heads/4.2 from Sowmya Krishnan
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=cfffe63 ]

CLOUDSTACK-3697 LB rule clean up already done in cleanup project


- ASF Subversion and Git Services


On July 23, 2013, 4:01 p.m., Sowmya Krishnan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12864/
> -----------------------------------------------------------
> 
> (Updated July 23, 2013, 4:01 p.m.)
> 
> 
> Review request for cloudstack and Prasanna Santhanam.
> 
> 
> Bugs: CLOUDSTACK-3697
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> LB is already deleted when project is cleaned up. Removed it from cleanup queue
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_project_resources.py 378238e 
> 
> Diff: https://reviews.apache.org/r/12864/diff/
> 
> 
> Testing
> -------
> 
> Tested script with fix locally and it passes now.
> 
> 
> Thanks,
> 
> Sowmya Krishnan
> 
>


Re: Review Request 12864: CLOUDSTACK-3697 Fix cleanup LB failure

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/12864/#review23720
-----------------------------------------------------------


Commit c6ea21866e008c467afdcc8b5d505c1c7978e164 in branch refs/heads/master from Sowmya Krishnan
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=c6ea218 ]

CLOUDSTACK-3697 LB rule clean up already done in cleanup project


- ASF Subversion and Git Services


On July 23, 2013, 4:01 p.m., Sowmya Krishnan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12864/
> -----------------------------------------------------------
> 
> (Updated July 23, 2013, 4:01 p.m.)
> 
> 
> Review request for cloudstack and Prasanna Santhanam.
> 
> 
> Bugs: CLOUDSTACK-3697
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> LB is already deleted when project is cleaned up. Removed it from cleanup queue
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_project_resources.py 378238e 
> 
> Diff: https://reviews.apache.org/r/12864/diff/
> 
> 
> Testing
> -------
> 
> Tested script with fix locally and it passes now.
> 
> 
> Thanks,
> 
> Sowmya Krishnan
> 
>


Re: Review Request 12864: CLOUDSTACK-3697 Fix cleanup LB failure

Posted by "Jenkins Cloudstack.org" <hu...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12864/#review23706
-----------------------------------------------------------


Review 12864 PASSED the build test
The url of build cloudstack-master-with-patch #24 is : http://jenkins.cloudstack.org/job/cloudstack-master-with-patch/24/

- Jenkins Cloudstack.org


On July 23, 2013, 4:01 p.m., Sowmya Krishnan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12864/
> -----------------------------------------------------------
> 
> (Updated July 23, 2013, 4:01 p.m.)
> 
> 
> Review request for cloudstack and Prasanna Santhanam.
> 
> 
> Bugs: CLOUDSTACK-3697
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> LB is already deleted when project is cleaned up. Removed it from cleanup queue
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_project_resources.py 378238e 
> 
> Diff: https://reviews.apache.org/r/12864/diff/
> 
> 
> Testing
> -------
> 
> Tested script with fix locally and it passes now.
> 
> 
> Thanks,
> 
> Sowmya Krishnan
> 
>


Re: Review Request 12864: CLOUDSTACK-3697 Fix cleanup LB failure

Posted by Sheng Yang <sh...@yasker.org>.

> On July 23, 2013, 8:55 p.m., Sheng Yang wrote:
> > Ship It!

Pushed to 4.2 and master.


- Sheng


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


On July 23, 2013, 4:01 p.m., Sowmya Krishnan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12864/
> -----------------------------------------------------------
> 
> (Updated July 23, 2013, 4:01 p.m.)
> 
> 
> Review request for cloudstack and Prasanna Santhanam.
> 
> 
> Bugs: CLOUDSTACK-3697
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> LB is already deleted when project is cleaned up. Removed it from cleanup queue
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_project_resources.py 378238e 
> 
> Diff: https://reviews.apache.org/r/12864/diff/
> 
> 
> Testing
> -------
> 
> Tested script with fix locally and it passes now.
> 
> 
> Thanks,
> 
> Sowmya Krishnan
> 
>


Re: Review Request 12864: CLOUDSTACK-3697 Fix cleanup LB failure

Posted by Sheng Yang <sh...@yasker.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12864/#review23715
-----------------------------------------------------------

Ship it!


Ship It!

- Sheng Yang


On July 23, 2013, 4:01 p.m., Sowmya Krishnan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12864/
> -----------------------------------------------------------
> 
> (Updated July 23, 2013, 4:01 p.m.)
> 
> 
> Review request for cloudstack and Prasanna Santhanam.
> 
> 
> Bugs: CLOUDSTACK-3697
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> LB is already deleted when project is cleaned up. Removed it from cleanup queue
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_project_resources.py 378238e 
> 
> Diff: https://reviews.apache.org/r/12864/diff/
> 
> 
> Testing
> -------
> 
> Tested script with fix locally and it passes now.
> 
> 
> Thanks,
> 
> Sowmya Krishnan
> 
>