like that it still doesn't apply (on 4.4, it does on 4.3) On Wed, Nov 19, 2014 at 11:32 AM, Anshul Gangwar <anshul.gang...@citrix.com> wrote:
> This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28213/ > > On November 19th, 2014, 9:35 a.m. UTC, *daan Hoogland* wrote: > > the patch does not apply to 4.4, Can you have a look? > > On November 19th, 2014, 10:01 a.m. UTC, *daan Hoogland* wrote: > > I tried applying to 4.3 as well. it failed too > > The error seems to be due to line endings in file. Existing file contains > windows line endings while patch has unix line endings. > > It is applying cleanly with --ignore-whitespace option. > > git am --ignore-whitespace [patchfile] > > > - Anshul > > On November 19th, 2014, 5:17 a.m. UTC, Anshul Gangwar wrote: > Review request for cloudstack, daan Hoogland and Rajesh Battala. > By Anshul Gangwar. > > *Updated Nov. 19, 2014, 5:17 a.m.* > *Bugs: * CLOUDSTACK-7752 > <https://issues.apache.org/jira/browse/CLOUDSTACK-7752> > *Repository: * cloudstack-git > Description > > Fixed deployment planner stuck in infinite loop. If we create VM with shared > service offering and attach disk with local disk offering, and one of storage > pool is full(cannot be allocated) and other is not full then we are not > putting the cluster in avoid list which is causing this infinite loop. > > Fixed by putting the cluster in avoid list even if one of the storage pool is > full(cannot be allocated) > > Diffs > > - server/src/com/cloud/deploy/DeploymentPlanningManagerImpl.java > (d9e1ecc2291a72a04ee0c1b35a079d5ff166bf77) > > View Diff <https://reviews.apache.org/r/28213/diff/> > -- Daan
