<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Hi Mike,<div><br></div><div>Thank you for catching this. This topic was discussed during the September PTG and it was agreed that: "The OVO sub-team has accepted to also continue the adoption of the new engine facade (<a href="https://blueprints.launchpad.net/neutron/+spec/enginefacade-switch">https://blueprints.launchpad.net/neutron/+spec/enginefacade-switch</a>), since both efforts work in the DB layer and entail traversing the code looking for opportunities of adoption" [1]. Unfortunately, we lost members of the OVO sub-team, to the point that the weekly meeting was recently removed [2]. As a consequence, this will be my personal goal for the Train cycle. I have restored the two patches that you pointed out and I'll re-start the effort from there [3][4]. I am also adding this topic to the discussion during the upcoming PTG.</div><div><br></div><div><br></div><div>[1] See section "Neutron upgrades/OVO" in <a href="http://lists.openstack.org/pipermail/openstack-dev/2018-September/135032.html">http://lists.openstack.org/pipermail/openstack-dev/2018-September/135032.html</a></div><div>[2] <a href="https://review.openstack.org/#/c/645181/">https://review.openstack.org/#/c/645181/</a></div><div>[3] <a href="https://review.openstack.org/#/c/545501/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/545501/</a><br>[4] <a href="https://review.openstack.org/#/c/396121/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/396121/</a><br></div></div></div></div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 16, 2019 at 8:13 AM Mike Bayer <<a href="mailto:mike_mp@zzzcomputing.com">mike_mp@zzzcomputing.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">There are a lot of very important and large DB related changes here<br>
that were unfortunately abandoned by their contributors [1] [2].<br>
Neutron remains from a SQLAlchemy / oslo.db perspective in this very<br>
awkward state where it is halfway between using legacy oslo.db access<br>
patterns and the "newer" enginefacade, which is now four years old.<br>
This leads to complicated issues that are difficult to develop around<br>
and debug [3].<br>
<br>
I don't have a solution to this problem as I am not tasked with<br>
Neutron to the degree that would be necessary to complete these<br>
migrations on my own, however as always, I eagerly await the arrival<br>
of developers who can commit to seeing these refactorings through,<br>
where I will of course be available to help.<br>
<br>
[1] <a href="https://review.openstack.org/#/c/545501/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/545501/</a><br>
[2] <a href="https://review.openstack.org/#/c/396121/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/396121/</a><br>
[3] <a href="https://review.openstack.org/#/c/348080/57/neutron/tests/unit/extensions/test_subnet_onboard.py@42" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/348080/57/neutron/tests/unit/extensions/test_subnet_onboard.py@42</a><br>
<br>
<br>
On Tue, Apr 16, 2019 at 5:08 AM Slawomir Kaplonski <<a href="mailto:skaplons@redhat.com" target="_blank">skaplons@redhat.com</a>> wrote:<br>
><br>
> Hi,<br>
><br>
> I just run abandon_old_reviews.sh [1] script again for neutron, neutron-lib and neutron-tempest-plugin repositories.<br>
> So if You have Your patch abandoned but You want to continue to work on it, please simple restore it or if You can’t restore it by self, please ping me on irc (slaweq) and I will restore it for You.<br>
><br>
> [1] <a href="https://github.com/openstack/neutron/blob/master/tools/abandon_old_reviews.sh" rel="noreferrer" target="_blank">https://github.com/openstack/neutron/blob/master/tools/abandon_old_reviews.sh</a><br>
><br>
> —<br>
> Slawek Kaplonski<br>
> Senior software engineer<br>
> Red Hat<br>
><br>
><br>
<br>
</blockquote></div>