[openstack-dev] [neutron] alembic 0.7.1 will break neutron's "heal" feature which assumes a fixed set of potential autogenerate types

Mike Bayer mbayer at redhat.com
Wed Dec 3 22:17:10 UTC 2014


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.

So it is now out, and the builds are failing due to https://launchpad.net/bugs/1397796 <https://launchpad.net/bugs/1397796>.

There’s some nits happening on the review  https://review.openstack.org/#/c/137989/ <https://review.openstack.org/#/c/137989/>, 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. 

- mike



> On Dec 1, 2014, at 5:43 PM, Salvatore Orlando <sorlando at nicira.com> wrote:
> 
> Thanks Mike!
> 
> I've left some comments on the patch.
> 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]?
> 
> Salvatore
> 
> [1] 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> 
> 
> On 1 December 2014 at 20:35, Mike Bayer <mbayer at redhat.com <mailto:mbayer at redhat.com>> wrote:
> hey neutron -
> 
> Just an FYI, I’ve added https://review.openstack.org/#/c/137989/ <https://review.openstack.org/#/c/137989/> / https://launchpad.net/bugs/1397796 <https://launchpad.net/bugs/1397796> 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.
> 
> 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.
> 
> Just a heads up in case you start seeing builds failing!
> 
> - mike
> 
> 
> 
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org <mailto:OpenStack-dev at lists.openstack.org>
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev <http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev>
> 
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20141203/5cc4d64b/attachment.html>


More information about the OpenStack-dev mailing list