Replying to list for visibility On Tue, 28 May 2019 at 19:31, Matthew Thode <mthode@mthode.org> wrote:

so, there are no requirements type reasons why we can't merge this now, but functional tests seem to be failing for nova/neutron (tempest). Would those teams be able to test?

https://review.opendev.org/651591 is the test patch (now updated for 1.3.4)

Some failures show up here as well.

http://logs.openstack.org/39/661539/1/check/tempest-full/b127828/testr_resul...

Thanks for filling the neutron-lib new version tag [0], it should be indeed enough for sqlalchemy 1.3 support (thanks to ralonsoh) For interested folks, [1] is the relevant fix in neutron-lib [0] https://review.opendev.org/#/c/661839 <https://review.opendev.org/#/c/661839/2>/ [1] https://review.opendev.org/#/c/651584/ -- Bernard Cafarelli