<div dir="ltr">Thanks Emilien and Ben. The situation is exactly as what Ben mentioned. The feature patches have been merged in Pike. But it is broken and we are trying to fix it. If we can backport it to Pike where the feature has been originally introduced, that would be great.<div><br><div>Here are the patches that cherry-picked to Pike:</div><div><a href="https://review.openstack.org/#/c/500946/">https://review.openstack.org/#/c/500946/</a><br></div><div><a href="https://review.openstack.org/#/c/500794/">https://review.openstack.org/#/c/500794/</a><br></div><div><a href="https://review.openstack.org/#/c/500674/">https://review.openstack.org/#/c/500674/</a><br></div><div><br></div><div>Thanks,</div><div>Tong</div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 5, 2017 at 10:32 AM, Emilien Macchi <span dir="ltr"><<a href="mailto:emilien@redhat.com" target="_blank">emilien@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Tue, Sep 5, 2017 at 10:16 AM, Ben Nemec <<a href="mailto:openstack@nemebean.com">openstack@nemebean.com</a>> wrote:<br>
><br>
><br>
> On 09/04/2017 11:50 PM, Emilien Macchi wrote:<br>
>><br>
>> On Fri, Sep 1, 2017 at 2:37 PM, Tong Liu <<a href="mailto:lexuns@gmail.com">lexuns@gmail.com</a>> wrote:<br>
>>><br>
>>> Hi,<br>
>>><br>
>>> In pike release, we added supported for Neutron NSX driver in TripleO [1]<br>
>>> by<br>
>>> the following patches. This will enable TripleO overcloud deployment to<br>
>>> use<br>
>>> vmware_nsx as Neutron core_plugin.<br>
>>> <a href="https://review.openstack.org/#/c/441668/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/441668/</a><br>
>>> <a href="https://review.openstack.org/#/c/452047/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/452047/</a><br>
>>> <a href="https://review.openstack.org/#/c/452391/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/452391/</a><br>
>>><br>
>>> However, there are some critical issues which prevent it from functional<br>
>>> correctly, and we fixed them in master with the following patches.<br>
>>> <a href="https://review.openstack.org/#/c/499395/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/499395/</a><br>
>>> <a href="https://review.openstack.org/#/c/498143/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/498143/</a><br>
>>> <a href="https://review.openstack.org/#/c/498142/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/498142/</a><br>
>>><br>
>>> Can we merged these patches and back port to pike?<br>
>><br>
>><br>
>> So if we follow all the rules, it goes against how to we handle stable<br>
>> branches and release management in general.<br>
><br>
><br>
> I don't know that I agree.  These patches are all for a feature that merged<br>
> earlier in Pike, but apparently is broken.  The only real issue I see is<br>
> that the first one doesn't have a bug reference and thus isn't valid for<br>
> backport as-is.  That one might need further discussion, but the others seem<br>
> to be legitimate bug fixes.<br>
><br>
> I also don't see any blueprint references, so I'm not sure how that came<br>
> into the discussion.  Maybe some confusion about the process for FFE's vs.<br>
> bugs?  It doesn't appear to me that a bp should be needed for these changes,<br>
> but maybe I'm missing something.<br>
<br>
</div></div>In that case let's threat them as bug fixes and let's do the backports<br>
to stable/pike.<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
>><br>
>> Usually what happens is we discuss about blueprints at PTG and then<br>
>> during the cycle developers implement the blueprints.<br>
>> Sometimes some blueprint get deferred for some reason (quite often<br>
>> it's because of overcommit but that's a separated topic) so they can<br>
>> as for FFE (feature freeze exception), granted by the PTL.<br>
>><br>
>> In your case, it's a bit more complex. You created the blueprint a few<br>
>> days before final release of Pike and you're asking us to merge the<br>
>> code AND backport it to Pike.<br>
>> That's for the facts & context, hope it helps to understand why this<br>
>> request is tricky.<br>
>><br>
>> Now you're a vendor and you're helping to support your driver, which is<br>
>> good.<br>
>><br>
>> We'll need to evaluate each commit and see if they are backward<br>
>> compatible and actually don't beak any interface (because we want our<br>
>> stable branches stable).<br>
>><br>
>> I'm ok to make an exception if next time you can do a better job in<br>
>> scheduling the work that will be done during one cycle.<br>
>> The way we propose blueprint is really lightweight, and in the open,<br>
>> so really no complication here.<br>
>><br>
>><br>
>> For now, most of the team is quite busy on Pike release (and PTG<br>
>> coming next week), so I'm not sure your patches will be reviewed soon<br>
>> (if yes, that's good).<br>
>> For the backports, we'll have to evaluate case by case and see if it's<br>
>> possible.<br>
>><br>
>> Thanks for your work and we hope our collaboration can happen earlier<br>
>> the next time.<br>
>><br>
>>> Thanks,<br>
>>> Tong<br>
>>> [1] Blueprint:<br>
>>> <a href="https://blueprints.launchpad.net/tripleo/+spec/neutron-nsx-driver" rel="noreferrer" target="_blank">https://blueprints.launchpad.<wbr>net/tripleo/+spec/neutron-nsx-<wbr>driver</a><br>
>><br>
>><br>
>><br>
><br>
<br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Emilien Macchi<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
______________________________<wbr>______________________________<wbr>______________<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.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
</div></div></blockquote></div><br></div>