migration error: invalid version number '0.7.3.dev'

Bug #1251700 reported by Dan Prince
20
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Critical
aeva black
OpenStack Compute (nova)
Fix Released
Critical
Dan Prince
oslo-incubator
Fix Released
Undecided
ChangBo Guo(gcb)
tuskar
Fix Released
High
Viktor Serhieiev

Bug Description

Using a tripleO seed VM I hit this issue today when trying to run the nova db migrations:

(nova)[root@localhost migrate]# /opt/stack/venvs/nova/bin/nova-manage --debug --verbose db sync
Command failed, please check log for more info
2013-11-15 16:53:18,579.579 9082 CRITICAL nova [-] invalid version number '0.7.3.dev'
2013-11-15 16:53:18,579.579 9082 TRACE nova Traceback (most recent call last):
2013-11-15 16:53:18,579.579 9082 TRACE nova File "/opt/stack/venvs/nova/bin/nova-manage", line 10, in <module>
2013-11-15 16:53:18,579.579 9082 TRACE nova sys.exit(main())
2013-11-15 16:53:18,579.579 9082 TRACE nova File "/opt/stack/venvs/nova/lib/python2.7/site-packages/nova/cmd/manage.py", line 1378, in main
2013-11-15 16:53:18,579.579 9082 TRACE nova ret = fn(*fn_args, **fn_kwargs)
2013-11-15 16:53:18,579.579 9082 TRACE nova File "/opt/stack/venvs/nova/lib/python2.7/site-packages/nova/cmd/manage.py", line 886, in sync
2013-11-15 16:53:18,579.579 9082 TRACE nova return migration.db_sync(version)
2013-11-15 16:53:18,579.579 9082 TRACE nova File "/opt/stack/venvs/nova/lib/python2.7/site-packages/nova/db/migration.py", line 31, in db_sync
2013-11-15 16:53:18,579.579 9082 TRACE nova return IMPL.db_sync(version=version)
2013-11-15 16:53:18,579.579 9082 TRACE nova File "/opt/stack/venvs/nova/lib/python2.7/site-packages/nova/utils.py", line 438, in __getattr__
2013-11-15 16:53:18,579.579 9082 TRACE nova backend = self.__get_backend()
2013-11-15 16:53:18,579.579 9082 TRACE nova File "/opt/stack/venvs/nova/lib/python2.7/site-packages/nova/utils.py", line 434, in __get_backend
2013-11-15 16:53:18,579.579 9082 TRACE nova self.__backend = __import__(name, None, None, fromlist)
2013-11-15 16:53:18,579.579 9082 TRACE nova File "/opt/stack/venvs/nova/lib/python2.7/site-packages/nova/db/sqlalchemy/migration.py", line 52, in <module>
2013-11-15 16:53:18,579.579 9082 TRACE nova dist_version.StrictVersion(migrate.__version__) < MIN_PKG_VERSION):
2013-11-15 16:53:18,579.579 9082 TRACE nova File "/usr/lib64/python2.7/distutils/version.py", line 40, in __init__
2013-11-15 16:53:18,579.579 9082 TRACE nova self.parse(vstring)
2013-11-15 16:53:18,579.579 9082 TRACE nova File "/usr/lib64/python2.7/distutils/version.py", line 107, in parse
2013-11-15 16:53:18,579.579 9082 TRACE nova raise ValueError, "invalid version number '%s'" % vstring
2013-11-15 16:53:18,579.579 9082 TRACE nova ValueError: invalid version number '0.7.3.dev'

Dan Prince (dan-prince)
Changed in nova:
assignee: nobody → Dan Prince (dan-prince)
status: New → In Progress
Changed in tripleo:
assignee: nobody → Dan Prince (dan-prince)
no longer affects: nova
Changed in tripleo:
status: New → In Progress
Revision history for this message
Dan Prince (dan-prince) wrote :

Okay. This just looks like a bug in the recently released https://pypi.python.org/pypi/sqlalchemy-migrate/0.8.1.

The __init__.py there as 0.7.3.dev coded as the version (incorrect).

Revision history for this message
Dan Prince (dan-prince) wrote :
Changed in nova:
assignee: nobody → Dan Prince (dan-prince)
importance: Undecided → Critical
status: New → In Progress
Julien Danjou (jdanjou)
Changed in ceilometer:
status: New → Confirmed
assignee: nobody → Julien Danjou (jdanjou)
importance: Undecided → Critical
Julien Danjou (jdanjou)
Changed in ceilometer:
assignee: Julien Danjou (jdanjou) → nobody
Revision history for this message
David Peraza (dperaza) wrote :

I think that for ceilometer stable/havana we should lock it to be less than 0.8.1. I'm going to send a patch to give that a try.

Changed in oslo:
status: New → Confirmed
aeva black (tenbrae)
Changed in ironic:
status: New → Confirmed
importance: Undecided → Critical
assignee: nobody → Devananda van der Veen (devananda)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic (master)

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

Changed in ironic:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

Reviewed: https://review.openstack.org/56705
Committed: http://github.com/openstack/ironic/commit/a5b2820686417b48a1e5e477b4aaeed159d770b1
Submitter: Jenkins
Branch: master

commit a5b2820686417b48a1e5e477b4aaeed159d770b1
Author: Devananda van der Veen <email address hidden>
Date: Fri Nov 15 14:52:57 2013 -0800

    Remove sqlalchemy-migrate 0.7.3 patching

    Now that sqlalchemy-migrate 0.8.x is available, remove the 0.7.3
    patching. This allows environments to pick up 0.8.1 automatically, while
    avoiding the version string error. Once an update to global-requirements
    is made, another patch will pin our sqlalchemy-migrate version >= 0.8.2.

    This is copied from change I5f80dfc07f8256716d025002fc1a1f47e58b2142

    Change-Id: I646a5a8f427095e20b09fbf2db796d2644def6fc
    Closes-bug: 1251700

Changed in ironic:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to oslo-incubator (master)

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

Changed in oslo:
assignee: nobody → ChangBo Guo (guochbo)
status: Confirmed → In Progress
Changed in tuskar:
assignee: nobody → Victor Sergeyev (vsergeyev)
status: New → Triaged
importance: Undecided → High
Revision history for this message
Roman Podoliaka (rpodolyaka) wrote :

This should not be critical as version 0.8.2 has been released, but it would be nice to clean all the projects that use sqlalchemy-migrate nevertheless.

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

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

Changed in tuskar:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tuskar (master)

Reviewed: https://review.openstack.org/56874
Committed: http://github.com/openstack/tuskar/commit/0fcd3ac3a62edf685e38484d72d67c613d918499
Submitter: Jenkins
Branch: master

commit 0fcd3ac3a62edf685e38484d72d67c613d918499
Author: Victor Sergeyev <email address hidden>
Date: Mon Nov 18 09:50:25 2013 +0200

    Remove sqlalchemy-migrate 0.7.3 patching

    Now that sqlalchemy-migrate 0.8.x is available, remove the 0.7.3
    patching. This allows environments to pick up 0.8.1 automatically, while
    avoiding the version string error. Once an update to global-requirements
    is made, another patch will pin our sqlalchemy-migrate version >= 0.8.2.

    This is copied from change I5f80dfc07f8256716d025002fc1a1f47e58b2142

    Change-Id: I646a5a8f427095e20b09fbf2db796d2644def6fc
    Closes-bug: 1251700

Changed in tuskar:
status: In Progress → Fix Committed
no longer affects: tripleo
Julien Danjou (jdanjou)
no longer affects: ceilometer
Thierry Carrez (ttx)
Changed in nova:
milestone: none → icehouse-1
Eric Harney (eharney)
no longer affects: cinder
Revision history for this message
Bhuvan Arumugam (bhuvan) wrote :

for the record we don't face this problem after sqlalchemy-migrate 0.8.2 is pushed to pypi.
http://pypi.openstack.org/openstack/sqlalchemy-migrate/sqlalchemy-migrate-0.8.2.tar.gz

Changed in tuskar:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

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

commit e26fac5ed7c7603b639aa1d93447feec96e0cda9
Author: Dan Prince <email address hidden>
Date: Fri Nov 15 12:14:57 2013 -0500

    Bump to sqlalchemy-migrate 0.8.2

    This allows us to drop some unsightly
    0.7.3 patching and also avoids potential
    issues in using 0.8.1 which had a bad
    release version string (it was called '0.7.3dev' by mistake).

    This will require a requirements change here:

     https://review.openstack.org/56662

    Change-Id: I5f80dfc07f8256716d025002fc1a1f47e58b2142
    Closes-Bug: 1251700

Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ironic:
milestone: none → icehouse-1
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to oslo-incubator (master)

Reviewed: https://review.openstack.org/56731
Committed: https://git.openstack.org/cgit/openstack/oslo-incubator/commit/?id=6603e8fe90da226af579b27ee364aa4bd89d57da
Submitter: Jenkins
Branch: master

commit 6603e8fe90da226af579b27ee364aa4bd89d57da
Author: Chang Bo Guo <email address hidden>
Date: Fri Nov 15 18:14:52 2013 -0800

    Remove sqlalchemy-migrate 0.7.3 patching

    Now that sqlalchemy-migrate 0.8.x is available, remove the 0.7.3
    patching. This allows environments to pick up 0.8.1 automatically,
    while avoiding the version string error. Once an update to
    global-requirements is made, another patch will pin our
    sqlalchemy-migrate version >= 0.8.2.

    This is copied from change I646a5a8f427095e20b09fbf2db796d2644def6fc

    Closes-bug: 1251700
    Change-Id: I51178d5b4dff56b92d2de08a73924bba56e975b6

Changed in oslo:
status: In Progress → Fix Committed
Changed in oslo:
milestone: none → icehouse-2
Thierry Carrez (ttx)
Changed in oslo:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in oslo:
milestone: icehouse-2 → 2014.1
Thierry Carrez (ttx)
Changed in nova:
milestone: icehouse-1 → 2014.1
Thierry Carrez (ttx)
Changed in ironic:
milestone: icehouse-1 → 2014.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.