<div dir="ltr">Thanks Slawek.  Are you planning to leave this change in place?  I can update my plugin's code, but that still leaves the likelihood of breakage if<div>- there's a new Rocky patch release</div><div>- a deployer is using an out-of-tree plugin with its own interface driver, and upgrades to the Rocky patch release</div><div>- either they don't also upgrade their plugin code, or there isn't a plugin update available because the plugin author hasn't noticed this problem yet.</div><div><br></div><div>Do you know if there will be another Rocky patch release, and if so when?</div><div><br></div><div>Best wishes,</div><div>    Neil</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, May 15, 2020 at 4:17 PM Slawek Kaplonski <<a href="mailto:skaplons@redhat.com">skaplons@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
That is my fault as I proposed this backport actually.<br>
I know it was mistake and we should do it a bit different in stable branches to<br>
avoid breaking third party drivers really.<br>
Maybe we should also think about moving such base driver classes to neutron-lib<br>
to avoid such issues in the future.<br>
<br>
Once again sorry for that. I will be more careful in the future.<br>
<br>
On Fri, May 15, 2020 at 03:33:13PM +0100, Neil Jerram wrote:<br>
> I'm sorry, but this is a moan.<br>
> <br>
> This merge -<br>
> <a href="https://opendev.org/openstack/neutron/commit/a6fb2faaa5d46656db9085ad6bcfc65ded807871" rel="noreferrer" target="_blank">https://opendev.org/openstack/neutron/commit/a6fb2faaa5d46656db9085ad6bcfc65ded807871</a><br>
> -<br>
> to the Neutron stable/rocky branch on April 23rd, has broken my team's<br>
> Neutron plugin, by requiring 3rd party LinuxInterfaceDriver subclasses to<br>
> take a new 'link_up' argument in their 'plug_new' method.<br>
> <br>
> IMO, it should have been obvious to folk proposing or reviewing this, that<br>
> it would cause breakage.<br>
> <br>
> Does Neutron have a different understanding of "stable" than I do?  Or do<br>
> plugins other than OVN not matter anymore?<br>
<br>
-- <br>
Slawek Kaplonski<br>
Senior software engineer<br>
Red Hat<br>
<br>
</blockquote></div>