Open Stack

Thanks Matt, will pester Nova-core when the time is right :)

-- dims

On Mon, Nov 17, 2014 at 1:57 PM, Matthew Treinish <mtreinish at kortar.org> wrote:
> On Mon, Nov 17, 2014 at 12:40:12PM -0500, Davanum Srinivas wrote:
>> Matt,
>>
>> Thanks for the help/reviews!. Looks like the sequence seems to be
>>
>> DevStack / Icehouse - https://review.openstack.org/#/c/134972/
>> DevStack / Juno - https://review.openstack.org/#/c/134975/
>> Tempest - https://review.openstack.org/#/c/134924/
>> Tempest - https://review.openstack.org/#/c/134985/
>> Nova - https://review.openstack.org/#/c/134332/
>>
>> Right?
>
> Yeah, that looks right.
>
> The only caveat is that before 134985 can land the procedure is that a Nova core
> has to +2 the 134332, and just leave a comment on the Tempest review saying it's
> got Nova core sign off with the link. (it's just procedural, I know that
> everyone wants to kill the XML API, myself included)
>
> -Matt Treinish
>
>> On Mon, Nov 17, 2014 at 8:48 AM, Matthew Treinish <mtreinish at kortar.org> wrote:
>> > On Mon, Nov 17, 2014 at 08:24:47AM -0500, Davanum Srinivas wrote:
>> >> Sean, Joe, Dean, all,
>> >>
>> >> Here's the Nova change to disable the V2 XML support:
>> >> https://review.openstack.org/#/c/134332/
>> >>
>> >> To keep all the jobs happy, we'll need changes in tempest, devstack,
>> >> devstack-gate as well:
>> >>
>> >> Tempest : https://review.openstack.org/#/c/134924/
>> >> Devstack : https://review.openstack.org/#/c/134685/
>> >> Devstack-gate : https://review.openstack.org/#/c/134714/
>> >>
>> >> Please see if i am on the right track.
>> >>
>> >
>> > So this approach will work, but the direction that neutron took and that
>> > keystone is in the process of undertaking for doing the same was basically the
>> > opposite. Instead of just overriding the default tempest value on master
>> > devstack to disable xml testing the devstack stable branches are update to
>> > ensure xml_api is True when running tempest from the stable branches, and then
>> > the default in tempest is switched to False. The advantage of this approach is
>> > that the default value for tempest running against any cloud will always work.
>> >
>> > The patches which landed for neutron doing this:
>> >
>> > Devstack:
>> > https://review.openstack.org/#/c/130368/
>> > https://review.openstack.org/#/c/130367/
>> >
>> > Tempest:
>> > https://review.openstack.org/#/c/127667/
>> >
>> > Neutron:
>> > https://review.openstack.org/#/c/128095/
>> >
>> >
>> > Ping me on IRC and we can work through the process, because things need to land
>> > in a particular order to make this approach work. But, basically the approach
>> > is first the stable devstack changes need to land which enable the testing on
>> > stable, followed by a +2 on the failing Nova patch saying the approach is
>> > approved, and then we can land the tempest patch which switches the default,
>> > which will let the Nova change get through the gate.
>> >
>> > -Matt Treinish
>> >
>> > _______________________________________________
>> > OpenStack-dev mailing list
>> > OpenStack-dev at lists.openstack.org
>> > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>> >
>>
>>
>>
>> --
>> Davanum Srinivas :: https://twitter.com/dims
>>
>> _______________________________________________
>> OpenStack-dev mailing list
>> OpenStack-dev at lists.openstack.org
>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>



-- 
Davanum Srinivas :: https://twitter.com/dims