[neutron] Recent incompatible change in stable/rocky branch
I'm sorry, but this is a moan. This merge - https://opendev.org/openstack/neutron/commit/a6fb2faaa5d46656db9085ad6bcfc65... - to the Neutron stable/rocky branch on April 23rd, has broken my team's Neutron plugin, by requiring 3rd party LinuxInterfaceDriver subclasses to take a new 'link_up' argument in their 'plug_new' method. IMO, it should have been obvious to folk proposing or reviewing this, that it would cause breakage. Does Neutron have a different understanding of "stable" than I do? Or do plugins other than OVN not matter anymore?
Hi, That is my fault as I proposed this backport actually. I know it was mistake and we should do it a bit different in stable branches to avoid breaking third party drivers really. Maybe we should also think about moving such base driver classes to neutron-lib to avoid such issues in the future. Once again sorry for that. I will be more careful in the future. On Fri, May 15, 2020 at 03:33:13PM +0100, Neil Jerram wrote:
I'm sorry, but this is a moan.
This merge - https://opendev.org/openstack/neutron/commit/a6fb2faaa5d46656db9085ad6bcfc65... - to the Neutron stable/rocky branch on April 23rd, has broken my team's Neutron plugin, by requiring 3rd party LinuxInterfaceDriver subclasses to take a new 'link_up' argument in their 'plug_new' method.
IMO, it should have been obvious to folk proposing or reviewing this, that it would cause breakage.
Does Neutron have a different understanding of "stable" than I do? Or do plugins other than OVN not matter anymore?
-- Slawek Kaplonski Senior software engineer Red Hat
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 - there's a new Rocky patch release - a deployer is using an out-of-tree plugin with its own interface driver, and upgrades to the Rocky patch release - 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. Do you know if there will be another Rocky patch release, and if so when? Best wishes, Neil On Fri, May 15, 2020 at 4:17 PM Slawek Kaplonski <skaplons@redhat.com> wrote:
Hi,
That is my fault as I proposed this backport actually. I know it was mistake and we should do it a bit different in stable branches to avoid breaking third party drivers really. Maybe we should also think about moving such base driver classes to neutron-lib to avoid such issues in the future.
Once again sorry for that. I will be more careful in the future.
On Fri, May 15, 2020 at 03:33:13PM +0100, Neil Jerram wrote:
I'm sorry, but this is a moan.
This merge -
- to the Neutron stable/rocky branch on April 23rd, has broken my team's Neutron plugin, by requiring 3rd party LinuxInterfaceDriver subclasses to take a new 'link_up' argument in their 'plug_new' method.
IMO, it should have been obvious to folk proposing or reviewing this,
https://opendev.org/openstack/neutron/commit/a6fb2faaa5d46656db9085ad6bcfc65... that
it would cause breakage.
Does Neutron have a different understanding of "stable" than I do? Or do plugins other than OVN not matter anymore?
-- Slawek Kaplonski Senior software engineer Red Hat
Hi, On Fri, May 15, 2020 at 06:20:07PM +0100, Neil Jerram wrote:
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 - there's a new Rocky patch release - a deployer is using an out-of-tree plugin with its own interface driver, and upgrades to the Rocky patch release - 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.
Can You maybe open Launchpad bug for that? It will be the same issue for all other stable branches like Stein or Train so we should fix it there too.
Do you know if there will be another Rocky patch release, and if so when?
Rocky is in EM phase now so we will not release it anymore.
Best wishes, Neil
On Fri, May 15, 2020 at 4:17 PM Slawek Kaplonski <skaplons@redhat.com> wrote:
Hi,
That is my fault as I proposed this backport actually. I know it was mistake and we should do it a bit different in stable branches to avoid breaking third party drivers really. Maybe we should also think about moving such base driver classes to neutron-lib to avoid such issues in the future.
Once again sorry for that. I will be more careful in the future.
On Fri, May 15, 2020 at 03:33:13PM +0100, Neil Jerram wrote:
I'm sorry, but this is a moan.
This merge -
- to the Neutron stable/rocky branch on April 23rd, has broken my team's Neutron plugin, by requiring 3rd party LinuxInterfaceDriver subclasses to take a new 'link_up' argument in their 'plug_new' method.
IMO, it should have been obvious to folk proposing or reviewing this,
https://opendev.org/openstack/neutron/commit/a6fb2faaa5d46656db9085ad6bcfc65... that
it would cause breakage.
Does Neutron have a different understanding of "stable" than I do? Or do plugins other than OVN not matter anymore?
-- Slawek Kaplonski Senior software engineer Red Hat
-- Slawek Kaplonski Senior software engineer Red Hat
On Fri, May 15, 2020 at 6:43 PM Slawek Kaplonski <skaplons@redhat.com> wrote:
Hi,
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 - there's a new Rocky patch release - a deployer is using an out-of-tree plugin with its own interface driver, and upgrades to the Rocky patch release - either they don't also upgrade their plugin code, or there isn't a
On Fri, May 15, 2020 at 06:20:07PM +0100, Neil Jerram wrote: plugin
update available because the plugin author hasn't noticed this problem yet.
Can You maybe open Launchpad bug for that? It will be the same issue for all other stable branches like Stein or Train so we should fix it there too.
Thanks Slawek, I've opened a bug here: https://bugs.launchpad.net/neutron/+bug/1879307 Best wishes, Neil
Hi, On Mon, May 18, 2020 at 12:15:05PM +0100, Neil Jerram wrote:
On Fri, May 15, 2020 at 6:43 PM Slawek Kaplonski <skaplons@redhat.com> wrote:
Hi,
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 - there's a new Rocky patch release - a deployer is using an out-of-tree plugin with its own interface driver, and upgrades to the Rocky patch release - either they don't also upgrade their plugin code, or there isn't a
On Fri, May 15, 2020 at 06:20:07PM +0100, Neil Jerram wrote: plugin
update available because the plugin author hasn't noticed this problem yet.
Can You maybe open Launchpad bug for that? It will be the same issue for all other stable branches like Stein or Train so we should fix it there too.
Thanks Slawek, I've opened a bug here: https://bugs.launchpad.net/neutron/+bug/1879307
Thank You. I just proposed patch https://review.opendev.org/729167 to address this issue. I will also propose backport of this patch to stable branches and we also agreed already that we will need to have this one merged before new release of the stable/Stein and stable/Train releases to not break others.
Best wishes, Neil
-- Slawek Kaplonski Senior software engineer Red Hat
participants (2)
-
Neil Jerram
-
Slawek Kaplonski