<div dir="ltr"><div dir="ltr">Replying to list for visibility</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 28 May 2019 at 19:31, Matthew Thode <<a href="mailto:mthode@mthode.org">mthode@mthode.org</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">so, there are no requirements type reasons why we can't merge this now,<br>
but functional tests seem to be failing for nova/neutron (tempest).<br>
Would those teams be able to test?<br>
<br>
<a href="https://review.opendev.org/651591" rel="noreferrer" target="_blank">https://review.opendev.org/651591</a> is the test patch (now updated for<br>
1.3.4)<br>
<br>
Some failures show up here as well.<br>
<br>
<a href="http://logs.openstack.org/39/661539/1/check/tempest-full/b127828/testr_results.html.gz" rel="noreferrer" target="_blank">http://logs.openstack.org/39/661539/1/check/tempest-full/b127828/testr_results.html.gz</a></blockquote><div><br></div><div>Thanks for filling the neutron-lib new version tag [0], it should be indeed enough for sqlalchemy 1.3 support (thanks to ralonsoh)</div><div>For interested folks, [1] is the relevant fix in neutron-lib<br></div><div><br></div><div>[0] <a href="https://review.opendev.org/#/c/661839/2">https://review.opendev.org/#/c/661839</a>/</div><div>[1] <a href="https://review.opendev.org/#/c/651584/">https://review.opendev.org/#/c/651584/</a></div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr">Bernard Cafarelli<br></div></div></div>