OpenStack Compute (nova)

SQLAlchemy (Migrate): useexisting is deprecated.

Bug #941951 reported by Christian Berendt
14
This bug affects 4 people
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Undecided
Joe Gordon
OpenStack Compute (nova) 2012.2 "folsom"

Bug Description

running nova-manage db sync I got the following Warning:

2012-02-27 14:35:24 WARNING nova.utils [-] /usr/lib64/python2.6/site-packages/nova/db/sqlalchemy/migrate_repo/versions/075_convert_bw_usage_to_store_network_id.py:49: SADeprecationWarning: useexisting is deprecated. Use extend_existing.
  useexisting=True)

using the following version of SQLAlchemy at the moment:

python-SQLAlchemy-0.7.5-3.1
python-sqlalchemy-migrate-0.7.2-2.1

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.openstack.org/4590

Changed in nova:
assignee: nobody → Chuck Short (zulcss)
status: New → In Progress
Revision history for this message
Hengqing Hu (hudayou) wrote :

That's for compatibility with sqlalchemy 0.6.x.
Not really a bug.
Similar warning exist in session.py, but for another option.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: master
Review: https://review.openstack.org/5081

Changed in nova:
assignee: Chuck Short (zulcss) → Pádraig Brady (p-draigbrady)
Revision history for this message
Pádraig Brady (p-draigbrady) wrote :

To confirm, this was deprecated in 0.7.0
http://hg.sqlalchemy.org/sqlalchemy/file/411362e11af2/CHANGES

We don't need compat with 0..6.x according to pip-requires,
also a similar change was just merged to glance:
https://review.openstack.org/#change,5031

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: master
Review: https://review.openstack.org/9785

Changed in nova:
assignee: Pádraig Brady (p-draigbrady) → Joe Gordon (joe-gordon0)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/9785
Committed: http://github.com/openstack/nova/commit/a834cf99f8a9b8b3fc9921da24a27f66e6abfbd1
Submitter: Jenkins
Branch: master

commit a834cf99f8a9b8b3fc9921da24a27f66e6abfbd1
Author: Joe Gordon <email address hidden>
Date: Fri Jul 13 03:58:33 2012 +0000

    Fix SADeprecationWarning: useexisting is deprecated

    Fix bug 941951

    Change-Id: I941df9260a4fbb27f5776f582ad8f59a1e32ea8b

Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → folsom-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: folsom-3 → 2012.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.