<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 16 November 2016 at 00:55, Gary Kotton <span dir="ltr"><<a href="mailto:gkotton@vmware.com" target="_blank">gkotton@vmware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div bgcolor="white" lang="EN-US">
<div class="m_305452894218498785gmail-m_-5424902409827100723m_-8857500227859734543WordSection1">
<p class="MsoNormal"><span style="font-size:11pt;font-family:calibri">Hi,<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:calibri">The directory integration will break all of the plugins and neutron projects. I do not think that this is something that we should do. It breaks the neutron API contract.</span></p></div></div></blockquote><div><br></div><div>The plugin directory is an implementation internal. Let's be very clear, in case you have not realized this already:</div><div><br></div><div><b>Neutron is not supposed to be imported directly by projects and we all knew it when we started off with the project decomposition.</b></div><div><br></div><div>neutron-lib is our response to driving adoption of stable interfaces across the neutron ecosystem of repositories. Forcing ourselves to introduce artificial deprecation cycles for internal details is not only slowing us down but it has proven ineffective so far. We should accelerate with the decoupling of projects so that we can all consider these types of breakages a thing of the past.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div bgcolor="white" lang="EN-US"><div class="m_305452894218498785gmail-m_-5424902409827100723m_-8857500227859734543WordSection1"><p class="MsoNormal"><span style="font-size:11pt;font-family:calibri"><u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:calibri">I think that we should only unblock the patch
<a href="https://review.openstack.org/#/c/386845" target="_blank">https://review.openstack.org/#<wbr>/c/386845</a>. I think that due to the fact that this patch (very big) will break all plugins, we should only approve it once every sub project owner has chimed in.</span> </p></div></div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div bgcolor="white" lang="EN-US"><div class="m_305452894218498785gmail-m_-5424902409827100723m_-8857500227859734543WordSection1"><p class="MsoNormal"><span style="font-size:11pt;font-family:calibri">This will mean
that she/he will need to understand that there may be some tweaks involved in getting unit tests to pass. CI may automagically work.</span></p></div></div></blockquote><div><br></div><div>This is impractical and defeats the point of allowing us to go faster. I have taken the proactive step of announcing this change publicly and with ample notice. I have addressed many subprojects myself and have already seen +2/+1 flocking in. I have moved forward without creating busy work for myself and the review team.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div bgcolor="white" lang="EN-US"><div class="m_305452894218498785gmail-m_-5424902409827100723m_-8857500227859734543WordSection1"><p class="MsoNormal"><span style="font-size:11pt;font-family:calibri"><u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:calibri">I feel that as a core reviewer my responsibility is to make sure that we do not break things.</span></p></div></div></blockquote><div><br></div><div>We are not in a sane situation. It's been two years since we split the repo up and very little progress has been made to decouple the projects via stable interfaces. I am trying to identify ways to allow us to accelerate and you're stifling that effort with your abuse of core rights. I was not going to let the patch merge without a final announcement at the next team meeting.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div bgcolor="white" lang="EN-US"><div class="m_305452894218498785gmail-m_-5424902409827100723m_-8857500227859734543WordSection1"><p class="MsoNormal"><span style="font-size:11pt;font-family:calibri"> In addition to this we have a responsibility to ensure that things continue to work. Hopefully
we can find a way to do this in a more friendly manner.</span></p></div></div></blockquote><div><br></div><div>I have taken such a responsibility with [1]. It takes us longer to discuss (on something that was already widely agreed on) than either fixing the breakage or provide a 'fake' backward compat layer which we'll lead to the breakage as soon we take it away [2].</div><div><br></div><div>That said, I am happy to concede if other members of the core team agrees with you. As PTL, I have identified a gap that needs to be filled and I am proactively stepping up to address the gap. I can't obviously be right all the time, but I was under the impression I had the majority of the core team on my side.</div><div><br></div><div>At this point, I'd invite other neutron core members to review and vote on the patch.</div><div><br></div><div>A.</div><div><br></div>[1] <a href="https://review.openstack.org/#/q/topic:plugin-directory" target="_blank">https://review.openstack.org/#<wbr>/q/topic:plugin-directory</a><br>[2] <a href="https://bugs.launchpad.net/vmware-nsx/+bug/1640319" target="_blank">https://bugs.launchpad.net/<wbr>vmware-nsx/+bug/1640319</a><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div bgcolor="white" lang="EN-US"><div class="m_305452894218498785gmail-m_-5424902409827100723m_-8857500227859734543WordSection1"><p class="MsoNormal"><span style="font-size:11pt;font-family:calibri"><u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:calibri">Thanks<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:calibri">Gary<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:calibri"><u></u> <u></u></span></p>
<div style="border-right:none;border-bottom:none;border-left:none;border-top:1pt solid rgb(181,196,223);padding:3pt 0in 0in">
<p class="MsoNormal"><b><span style="font-family:calibri;color:black">From: </span>
</b><span style="font-family:calibri;color:black">"Armando M." <<a href="mailto:armamig@gmail.com" target="_blank">armamig@gmail.com</a>><br>
<b>Reply-To: </b>OpenStack List <<a href="mailto:openstack-dev@lists.openstack.org" target="_blank">openstack-dev@lists.openstack<wbr>.org</a>><br>
<b>Date: </b>Wednesday, November 16, 2016 at 6:51 AM<br>
<b>To: </b>OpenStack List <<a href="mailto:openstack-dev@lists.openstack.org" target="_blank">openstack-dev@lists.openstack<wbr>.org</a>><br>
<b>Subject: </b>[openstack-dev] [neutron] neutron-lib impact<u></u><u></u></span></p>
</div><div><div class="m_305452894218498785gmail-m_-5424902409827100723h5">
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Hi neutrinos, <u></u><u></u></p>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">As mentioned during the last team meeting [1], there is a change [2] in the works aimed at adopting the neutron plugins directory as provided in neutron-lib 1.0.0 [3].<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">As shown in [2], the switch to using the directory is relatively straightforward. I leave the rest of the affected repos as an exercise for the reader :)<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Cheers,<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">Armando<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<div>
<p class="MsoNormal">[1] <a href="http://eavesdrop.openstack.org/meetings/networking/2016/networking.2016-11-14-21.00.txt" target="_blank">http://eavesdrop.openstack<wbr>.org/meetings/networking/2016/<wbr>networking.2016-11-14-21.00.tx<wbr>t</a><u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">[2] <a href="https://review.openstack.org/#/q/topic:plugin-directory" target="_blank">https://review.openstack.o<wbr>rg/#/q/topic:plugin-directory</a><u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">[3] <a href="http://docs.openstack.org/releasenotes/neutron-lib/unreleased.html#id3" target="_blank">http://docs.openstack.org/<wbr>releasenotes/neutron-lib/unrel<wbr>eased.html#id3</a><u></u><u></u></p>
</div>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
</div>
</div></div></div>
</div>
<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.op<wbr>enstack.org?subject:unsubscrib<wbr>e</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi<wbr>-bin/mailman/listinfo/openstac<wbr>k-dev</a><br>
<br></blockquote></div><br></div></div>