You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Wei Zhou <w....@leaseweb.com> on 2013/03/20 10:04:31 UTC
Review Request: remove duplicated ExpungeTask on 4.0 branch
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10042/
-----------------------------------------------------------
Review request for cloudstack.
Description
-------
There are two ExpungeTask running in cloudstack 4.0.1.
see https://issues.apache.org/jira/browse/CLOUDSTACK-1683
We need to remove duplicated ExpungeTask from BareMetalVmManagerImpl.
This patch only applies on 4.0.1, for the issue does not appear on master/4.1
This addresses bug CLOUDSTACK-1683.
Diffs
-----
server/src/com/cloud/baremetal/BareMetalVmManagerImpl.java 3972728
Diff: https://reviews.apache.org/r/10042/diff/
Testing
-------
ok.
Thanks,
Wei Zhou
Re: Review Request: remove duplicated ExpungeTask on 4.0 branch
Posted by Chip Childers <ch...@sungard.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10042/
-----------------------------------------------------------
(Updated March 20, 2013, 3:39 p.m.)
Review request for cloudstack and Joe Brockmeier.
Changes
-------
Added jzb, as this is for 4.0.2.
Adjusted the branch from 4.0.1 to 4.0 (since 4.0 is the correct branch name for all 4.0.x releases).
Description
-------
There are two ExpungeTask running in cloudstack 4.0.1.
see https://issues.apache.org/jira/browse/CLOUDSTACK-1683
We need to remove duplicated ExpungeTask from BareMetalVmManagerImpl.
This patch only applies on 4.0.1, for the issue does not appear on master/4.1
This addresses bug CLOUDSTACK-1683.
Diffs
-----
server/src/com/cloud/baremetal/BareMetalVmManagerImpl.java 3972728
Diff: https://reviews.apache.org/r/10042/diff/
Testing
-------
ok.
Thanks,
Wei Zhou