Re: [openstack-dev] [neutron] Re: [stable] Requesting exception for stable/kilo

2016-05-09 Thread Sukhdev Kapur
Hi Akihito,

I think from the requirements.txt point of view, we are good.

Thanks
-Sukhdev




On Mon, May 9, 2016 at 2:11 PM, Akihiro Motoki  wrote:

> Hi Sukhdev, Dave,
>
> I think this backport is acceptable.
>
> In Kilo release, neutron is in a middle way of splitting vendor codes
> to separate repositories.
> The master fix is available at https://review.openstack.org/#/c/309622/
> and
> as far as I checked, stable/kilo fix are split into two parts:
> - networking-arista (db_api code):
> https://review.openstack.org/#/c/309651/
> - neutron : https://review.openstack.org/#/c/309653/
> It satisfies the guideline of backport of split-out vendor code [1].
>
> One point I'd like to mention is that this change requires a new
> release of networking-arista
> because the change uses new method from networking-arista.
> Do you need to update the vendor requirements.txt file [2] ?
>
> Otherwise it looks good.
>
> Akihiro
>
> [1]
> http://docs.openstack.org/developer/neutron/devref/contribute.html#backport-management-strategies
> [2]
> https://github.com/openstack/neutron/blob/stable/kilo/neutron/plugins/ml2/drivers/arista/requirements.txt
>
> 2016-05-10 4:46 GMT+09:00 Dave Walker :
> >
> >
> > On 4 May 2016 at 17:55, Sukhdev Kapur  wrote:
> >>
> >> Hi Stable Maintainers,
> >>
> >> We have a critical bug impacting customers production network. This is a
> >> minor fix.
> >> I would like to request an exception so that the customers do not have
> to
> >> baby
> >> sit this patch.
> >>
> >> https://review.openstack.org/#/c/309653/
> >>
> >> Please allow this to be merged.
> >>
> >> Thanks
> >> -Sukhdev
> >>
> >
> > Hi,
> >
> > Thanks for raising this.
> >
> > I'm currently blocking the 2015.1.4 release on this request.  As it
> isn't a
> > clean backport and the 11th hour, I really want a review from
> neutron-core.
> >
> > However, this hasn't been forthcoming.  I really need this asap, or we
> will
> > have to release without it.
> >
> > Ps. apologies for the empty reply just now.
> >
> > --
> > Kind Regards,
> > Dave Walker
> >
> >
> __
> > OpenStack Development Mailing List (not for usage questions)
> > Unsubscribe:
> openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
> > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> >
>
> __
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


Re: [openstack-dev] [neutron] Re: [stable] Requesting exception for stable/kilo

2016-05-09 Thread Akihiro Motoki
Hi Sukhdev, Dave,

I think this backport is acceptable.

In Kilo release, neutron is in a middle way of splitting vendor codes
to separate repositories.
The master fix is available at https://review.openstack.org/#/c/309622/ and
as far as I checked, stable/kilo fix are split into two parts:
- networking-arista (db_api code): https://review.openstack.org/#/c/309651/
- neutron : https://review.openstack.org/#/c/309653/
It satisfies the guideline of backport of split-out vendor code [1].

One point I'd like to mention is that this change requires a new
release of networking-arista
because the change uses new method from networking-arista.
Do you need to update the vendor requirements.txt file [2] ?

Otherwise it looks good.

Akihiro

[1] 
http://docs.openstack.org/developer/neutron/devref/contribute.html#backport-management-strategies
[2] 
https://github.com/openstack/neutron/blob/stable/kilo/neutron/plugins/ml2/drivers/arista/requirements.txt

2016-05-10 4:46 GMT+09:00 Dave Walker :
>
>
> On 4 May 2016 at 17:55, Sukhdev Kapur  wrote:
>>
>> Hi Stable Maintainers,
>>
>> We have a critical bug impacting customers production network. This is a
>> minor fix.
>> I would like to request an exception so that the customers do not have to
>> baby
>> sit this patch.
>>
>> https://review.openstack.org/#/c/309653/
>>
>> Please allow this to be merged.
>>
>> Thanks
>> -Sukhdev
>>
>
> Hi,
>
> Thanks for raising this.
>
> I'm currently blocking the 2015.1.4 release on this request.  As it isn't a
> clean backport and the 11th hour, I really want a review from neutron-core.
>
> However, this hasn't been forthcoming.  I really need this asap, or we will
> have to release without it.
>
> Ps. apologies for the empty reply just now.
>
> --
> Kind Regards,
> Dave Walker
>
> __
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>

__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


Re: [openstack-dev] [neutron] Re: [stable] Requesting exception for stable/kilo

2016-05-09 Thread Sukhdev Kapur
Hi Dave,

I will get on it right away. Let me find Neutron Cores to review it.

Please stand by

-Sukhdev


On Mon, May 9, 2016 at 12:46 PM, Dave Walker  wrote:

>
>
> On 4 May 2016 at 17:55, Sukhdev Kapur  wrote:
>
>> Hi Stable Maintainers,
>>
>> We have a critical bug impacting customers production network. This is a
>> minor fix.
>> I would like to request an exception so that the customers do not have to
>> baby
>> sit this patch.
>>
>> https://review.openstack.org/#/c/309653/
>>
>> Please allow this to be merged.
>>
>> Thanks
>> -Sukhdev
>>
>>
> Hi,
>
> Thanks for raising this.
>
> I'm currently blocking the 2015.1.4 release on this request.  As it isn't
> a clean backport and the 11th hour, I really want a review from
> neutron-core.
>
> However, this hasn't been forthcoming.  I really need this asap, or we
> will have to release without it.
>
> Ps. apologies for the empty reply just now.
>
> --
> Kind Regards,
> Dave Walker
>
> __
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


[openstack-dev] [neutron] Re: [stable] Requesting exception for stable/kilo

2016-05-09 Thread Dave Walker
On 4 May 2016 at 17:55, Sukhdev Kapur  wrote:

> Hi Stable Maintainers,
>
> We have a critical bug impacting customers production network. This is a
> minor fix.
> I would like to request an exception so that the customers do not have to
> baby
> sit this patch.
>
> https://review.openstack.org/#/c/309653/
>
> Please allow this to be merged.
>
> Thanks
> -Sukhdev
>
>
Hi,

Thanks for raising this.

I'm currently blocking the 2015.1.4 release on this request.  As it isn't a
clean backport and the 11th hour, I really want a review from neutron-core.

However, this hasn't been forthcoming.  I really need this asap, or we will
have to release without it.

Ps. apologies for the empty reply just now.

--
Kind Regards,
Dave Walker
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


[openstack-dev] [neutron] Re: [stable] Requesting exception for stable/kilo

2016-05-09 Thread Dave Walker
On 4 May 2016 at 17:55, Sukhdev Kapur  wrote:

> Hi Stable Maintainers,
>
> We have a critical bug impacting customers production network. This is a
> minor fix.
> I would like to request an exception so that the customers do not have to
> baby
> sit this patch.
>
> https://review.openstack.org/#/c/309653/
>
> Please allow this to be merged.
>
> Thanks
> -Sukhdev
>
> __
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev