← Back to team overview

fuel-dev team mailing list archive

Re: 4.1.1 bugfixes ready for review/merge

 

Thanks Dmitry.

Ananstasia, is it something we know about and fixing? Do we have bug report
for it?

We need to fix this ASAP. It's critical for releasing maintenance build.


On Mon, Apr 7, 2014 at 2:41 AM, Dmitry Borodaenko
<dborodaenko@xxxxxxxxxxxx>wrote:

> Last I checked, a bunch of reviews targeted for stable/4.1 were stuck due
> to a CI problem, once that is cleared they will need to be rebased and
> merged. There's a list in my daily syncup report from Friday.
>  On Apr 6, 2014 11:29 AM, "Mike Scherbakov" <mscherbakov@xxxxxxxxxxxx>
> wrote:
>
>> Thanks folks.
>> Anything else we want to merge / backport? I'm expecting to have 4.1A no
>> later than Tuesday...
>>
>>
>> On Thu, Apr 3, 2014 at 8:44 PM, Dmitry Borodaenko <
>> dborodaenko@xxxxxxxxxxxx> wrote:
>>
>>> Well spotted, same change resubmitted with the original Change-Id
>>> here: https://review.openstack.org/85105
>>>
>>> On Thu, Apr 3, 2014 at 1:49 AM, Evgeniy L <eli@xxxxxxxxxxxx> wrote:
>>> > Hi Dmitry,
>>> >
>>> > This patches are same but they have different Change-Id:
>>> >
>>> > https://review.openstack.org/#/c/84203/
>>> > https://review.openstack.org/#/c/80689/
>>> >
>>> > I thought we agreed not to change Change-Id when we backport patches,
>>> am I
>>> > right?
>>> >
>>> > Thanks
>>> >
>>> >
>>> >
>>> > On Thu, Apr 3, 2014 at 2:27 AM, Dmitry Borodaenko <
>>> dborodaenko@xxxxxxxxxxxx>
>>> > wrote:
>>> >>
>>> >> FAO core reviewers:
>>> >>
>>> >> https://review.openstack.org/84544 (Subs 1.0.0.0 -> 198.18.1.[1|2] in
>>> >> network checker) -- backport of a fix by Vladimir Kozhukalov, needs to
>>> >> be reviewed (wasn't a trivial rebase) and merged into fuel-main
>>> >> stable/4.1
>>> >>
>>> >> https://review.openstack.org/84203 (do not create radosgw region map
>>> >> in post-deploy) -- backport of my fix for fuel-astute stable/4.1, also
>>> >> not a trivial rebase, but a very obvious change so should not be a
>>> >> problem
>>> >>
>>> >> https://review.openstack.org/82965 (Set Ceph pg_num dynamically based
>>> >> on fuel_settings) -- proposed for fuel-library master, will need a
>>> >> backport once merged, fuel-web counterpart is already merged in master
>>> >> and stable/4.1
>>> >>
>>> >> https://review.openstack.org/82629 (Use internal IP for internal and
>>> >> admin RGW endpoints) -- proposed for fuel-library master, will need a
>>> >> backport once merged, no external dependencies
>>> >>
>>> >> I'd rather not establish a bad precedent of summarily merging my own
>>> >> commits, but these fixes are harmless enough and should really land in
>>> >> stable/4.1 by the end of this week.
>>> >>
>>> >> Thanks,
>>> >>
>>> >> --
>>> >> Dmitry Borodaenko
>>> >>
>>> >> --
>>> >> Mailing list: https://launchpad.net/~fuel-dev
>>> >> Post to     : fuel-dev@xxxxxxxxxxxxxxxxxxx
>>> >> Unsubscribe : https://launchpad.net/~fuel-dev
>>> >> More help   : https://help.launchpad.net/ListHelp
>>> >
>>> >
>>>
>>>
>>>
>>> --
>>> Dmitry Borodaenko
>>>
>>> --
>>> Mailing list: https://launchpad.net/~fuel-dev
>>> Post to     : fuel-dev@xxxxxxxxxxxxxxxxxxx
>>> Unsubscribe : https://launchpad.net/~fuel-dev
>>> More help   : https://help.launchpad.net/ListHelp
>>>
>>
>>
>>
>> --
>> Mike Scherbakov
>> #mihgen
>>
>


-- 
Mike Scherbakov
#mihgen

Follow ups

References