fuel-dev team mailing list archive

[Dmitry Borodaenko <dborodaenko@xxxxxxxxxxxx>]

I've backported all RabbitMQ3 related fixes to stable/4.1:
https://review.openstack.org/#/c/87371/
https://review.openstack.org/#/c/87379/
https://review.openstack.org/#/c/87383/

However the fuel-web CI still seems to be broken for 4.1, it insists
on testing fuel_upgrade_system which doesn't exist in stable/4.1:
https://fuel-jenkins.mirantis.com/job/stackforge-verify-fuel-web/2932/console

Anyone knows how to fix that?

Thanks,
-DmitryB


On Mon, Apr 14, 2014 at 10:55 AM, Dmitry Borodaenko
<dborodaenko@xxxxxxxxxxxx> wrote:
> Unfortunately, even though both these reviews (85111 and 85112) were
> unblocked by CI last Monday, both were left unmerged for a week. I
> rebased both, CI jobs are still running but I don't expect any more
> breakage, rebase was trivial for both. Please merge when possible,
> it's really disappointing that these fixes didn't make it into
> stable/4.1 last week.
>
> Thanks,
> -DmitryB
>
> On Mon, Apr 7, 2014 at 3:35 AM, Mike Scherbakov
> <mscherbakov@xxxxxxxxxxxx> wrote:
>> Ok, so what's the reason of "-1" - is it because of non-working code or CI
>> issue?
>>
>>
>> On Mon, Apr 7, 2014 at 2:31 PM, Anastasia Urlapova <aurlapova@xxxxxxxxxxxx>
>> wrote:
>>>
>>> Mike,
>>> as I see, only two reviews with "-1"
>>> https://review.openstack.org/#/c/85111
>>> https://review.openstack.org/#/c/85112
>>>
>>> others are just in progress|confirmed.
>>>
>>>
>>> On Mon, Apr 7, 2014 at 1:20 PM, Mike Scherbakov <mscherbakov@xxxxxxxxxxxx>
>>> wrote:
>>>>
>>>> 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
>>>
>>>
>>
>>
>>
>> --
>> Mike Scherbakov
>> #mihgen
>
>
>
> --
> Dmitry Borodaenko



-- 
Dmitry Borodaenko