test suite doesn't work with sqlalchemy v7

Bug #892754 reported by Monty Taylor
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
Fix Released
Low
Andrew Hutchings

Bug Description

When we tried aligning the glance pip-requires with nova in terms of sqlalchemy versions, we got a bunch of funky errors from sqlalchemy (which was installing 7.0.3) These may just be how the test suite runs, as people seemed to think it should have worked.

In any case - I'd really like to get us aligned on versions there - so we should probably fix this.

Revision history for this message
Andrew Hutchings (linuxjedi) wrote :

Problem 1:
test_bad_sql_connection fails and breaks _ENGINE for the rest of the SQL Alchemy based tests

Problem 2:
test_walk_versions also breaks

Not figured out the cause of these two yet.

Changed in glance:
assignee: nobody → Andrew Hutchings (linuxjedi)
Revision history for this message
Andrew Hutchings (linuxjedi) wrote :

My own notes so far:

Problem 1 I have a patch for (going to attach here until I have set everything up to do workflow)

Problem 2 migration_api.downgrade(options, version) version 9->8 failing.

Revision history for this message
Andrew Hutchings (linuxjedi) wrote :
Jay Pipes (jaypipes)
Changed in glance:
status: New → Confirmed
importance: Undecided → Low
Changed in glance:
status: Confirmed → In Progress
Jay Pipes (jaypipes)
Changed in glance:
milestone: none → essex-4
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to glance (master)

Reviewed: https://review.openstack.org/3814
Committed: http://github.com/openstack/glance/commit/e4db87ce4c2757b734b5fc62a82e1076c3367af0
Submitter: Jenkins
Branch: master

commit e4db87ce4c2757b734b5fc62a82e1076c3367af0
Author: Andrew Hutchings <email address hidden>
Date: Mon Feb 6 18:35:49 2012 +0000

    Make Glance work with SQLAlchemy 0.7

    Fix execption caught during tests
    Fix test failure wiping out engine in use

    Fixes bug 892754

    Change-Id: I9de6f077af656730c95bc715c2b765b0c49c7d3c

Changed in glance:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in glance:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in glance:
milestone: essex-4 → 2012.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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