<div dir="ltr">Thanks Mike!<div><br></div><div>I've left some comments on the patch.</div><div>Just out of curiosity, since now alembic can autogenerate foreign keys, are we be able to remove the logic for identifying foreign keys to add/remove [1]?</div><div><br></div><div>Salvatore</div><div><br></div><div>[1] <a href="http://git.openstack.org/cgit/openstack/neutron/tree/neutron/db/migration/alembic_migrations/heal_script.py#n205">http://git.openstack.org/cgit/openstack/neutron/tree/neutron/db/migration/alembic_migrations/heal_script.py#n205</a> </div></div><div class="gmail_extra"><br><div class="gmail_quote">On 1 December 2014 at 20:35, Mike Bayer <span dir="ltr"><<a href="mailto:mbayer@redhat.com" target="_blank">mbayer@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">hey neutron -<br>
<br>
Just an FYI, I’ve added <a href="https://review.openstack.org/#/c/137989/" target="_blank">https://review.openstack.org/#/c/137989/</a> / <a href="https://launchpad.net/bugs/1397796" target="_blank">https://launchpad.net/bugs/1397796</a> to refer to an issue in neutron’s “heal” script that is going to start failing when I put out Alembic 0.7.1, which is potentially later today / this week.<br>
<br>
The issue is pretty straightforward,  Alembic 0.7.1 is adding foreign key autogenerate (and really, could add more types of autogenerate at any time), and as these new commands are revealed within the execute_alembic_command(), they are not accounted for, so it fails.   I’d recommend folks try to push this one through or otherwise decide how this issue (which should be expected to occur many more times) should be handled.<br>
<br>
Just a heads up in case you start seeing builds failing!<br>
<br>
- mike<br>
<br>
<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div><br></div>