<div dir="ltr">Thanks Mike,<div><br></div><div>I already gave your patch a +2. when the gate is broken there's no time for nitpicking - those can come with a followup patch.</div><div>The patch is now spinning again against jenkins checks. As soon as it's done we'll send it through the gate - and hopefully get a promotion for it.</div><div><br></div><div>Salvatore</div></div><div class="gmail_extra"><br><div class="gmail_quote">On 3 December 2014 at 23:17, 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"><div style="word-wrap:break-word">So folks, I had to put Alembic 0.7.1 out as I realized that the “batch” mode was being turned on for autogenerate across the board in 0.7.0, and that was not the plan.<div><br></div><div>So it is now out, and the builds are failing due to <a href="https://launchpad.net/bugs/1397796" target="_blank">https://launchpad.net/bugs/1397796</a>.</div><div><br></div><div>There’s some nits happening on the review  <a href="https://review.openstack.org/#/c/137989/" target="_blank">https://review.openstack.org/#/c/137989/</a>, so I’m hoping someone with some Neutron cred adjust the patch to their liking and get it merged.    I’m just the messenger on this. </div><span class="HOEnZb"><font color="#888888"><div><br></div><div>- mike</div></font></span><div><div class="h5"><div><br></div><div><br></div><div><br><div><blockquote type="cite"><div>On Dec 1, 2014, at 5:43 PM, Salvatore Orlando <<a href="mailto:sorlando@nicira.com" target="_blank">sorlando@nicira.com</a>> wrote:</div><br><div><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" target="_blank">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" target="_blank">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>
_______________________________________________<br>OpenStack-dev mailing list<br><a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">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></div></blockquote></div><br></div></div></div></div><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>
<br></blockquote></div><br></div>