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