"neutron-db-manage revision" complains about multiple heads

Bug #1503342 reported by vikram.choudhary
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
neutron
Fix Released
High
YujiAzama

Bug Description

openstack@nwbl-openstack:~/devstack$ neutron-db-manage revision -m "xxx"
  Running revision for neutron ...
  FAILED: Multiple heads are present; please specify the head revision on which the new revision should be
  based, or perform a merge.

Henry Gessau (gessau)
tags: added: db
summary: - Error observed while running "neutron-db-manage revision -m "my
- migration" "
+ "neutron-db-manage revision" complains about multiple heads
Changed in neutron:
importance: Undecided → High
status: New → Confirmed
assignee: nobody → Henry Gessau (gessau)
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

what's xxx some migration with X-rated content?

Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

:)

Revision history for this message
vikram.choudhary (vikschw) wrote :

Just for test ;)

Changed in neutron:
milestone: none → mitaka-1
Revision history for this message
changzhi (changzhi) wrote :

Does anyone fix it?

Revision history for this message
Henry Gessau (gessau) wrote :

Yes I am working on it.

Revision history for this message
YAMAMOTO Takashi (yamamoto) wrote :
Changed in neutron:
assignee: Henry Gessau (gessau) → YujiAzama (azama-yuji)
status: Confirmed → In Progress
Changed in neutron:
assignee: YujiAzama (azama-yuji) → Henry Gessau (gessau)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/223901
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=9d069c48aed3a087c5c51366c8e70b29f339e794
Submitter: Jenkins
Branch: master

commit 9d069c48aed3a087c5c51366c8e70b29f339e794
Author: Yuji <email address hidden>
Date: Wed Sep 16 11:45:27 2015 +0900

    Allow to specify branch for creating new migration

    Add options to 'neutron-db-manage revision' command for specifying
    a branch.

    DocImpact

    Change-Id: I00b00821839b0d77caaaf26b518c5c0c75e01218
    Closes-Bug: 1503342

Changed in neutron:
status: In Progress → Fix Committed
Henry Gessau (gessau)
Changed in neutron:
assignee: Henry Gessau (gessau) → YujiAzama (azama-yuji)
Revision history for this message
YujiAzama (azama-yuji) wrote :

Henry, Thank you very much:)

Henry Gessau (gessau)
tags: added: liberty-backport-potential
Revision history for this message
Miguel Angel Ajo (mangelajo) wrote :

I agree, we should backport this if possible.

Revision history for this message
Ann Taraday (akamyshnikova) wrote :

Just curious: Why should we backport this? In stable/liberty we can not create new migrations, so what is the case when it is needed?

Revision history for this message
Henry Gessau (gessau) wrote :

Ann, the revision command without --autogenerate should work without giving this error. Also I believe out-of-tree extensions that have not yet released their liberty version may want to use the --autogenerate.

Revision history for this message
Miguel Angel Ajo (mangelajo) wrote :

And eventually, we could want to be able to backport migrations if necessary.

Revision history for this message
Ann Taraday (akamyshnikova) wrote :

@Henry: OK
@Miguel: backporting migrations is a very odd thing and will require checks in migrations, so that it won't crash with upgrade after that. This should be a very serious issue to do such staff.

Revision history for this message
Miguel Angel Ajo (mangelajo) wrote :

@ann, I know it's hard, but eventually it could get easier with better support from alembic, not sure if branches already allow such thing... We better not needed, I agree...

Revision history for this message
Mike Bayer (zzzeek) wrote :

@mangelajo - what is "such thing" in this context? you can merge branches in Alembic, I believe Ann is referring to the migration check feature they've added to Neutron.

Revision history for this message
Dongcan Ye (hellochosen) wrote :

@All

If I had an extension need to rebase to Liberty, when running command:

neutron-db-manage revision -m "****" --autogenerate

it will failed as error:
FAILED: Multiple heads are present; please specify the head revision on which the new revision should be based, or perform a merge.

But in Liberty, neutron-db-manage wrapper does't support --expand or --contract.

Revision history for this message
Ihar Hrachyshka (ihar-hrachyshka) wrote :

I believe a use case to create a new branch that will work with neutron/Liberty is a good one to consider backports. Please propose and let's follow up in gerrit.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/liberty)

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/247987

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/liberty)

Reviewed: https://review.openstack.org/247987
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=d6a628e5453ed77af265c0a3c6cb2e1e6c0c22c9
Submitter: Jenkins
Branch: stable/liberty

commit d6a628e5453ed77af265c0a3c6cb2e1e6c0c22c9
Author: Yuji <email address hidden>
Date: Wed Sep 16 11:45:27 2015 +0900

    Allow to specify branch for creating new migration

    Add options to 'neutron-db-manage revision' command for specifying
    a branch.

    DocImpact

    Change-Id: I00b00821839b0d77caaaf26b518c5c0c75e01218
    Closes-Bug: 1503342
    (cherry picked from commit 9d069c48aed3a087c5c51366c8e70b29f339e794)

tags: added: in-stable-liberty
Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/neutron 8.0.0.0b1

This issue was fixed in the openstack/neutron 8.0.0.0b1 development milestone.

Changed in neutron:
status: Fix Committed → Fix Released
Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/neutron 7.0.1

This issue was fixed in the openstack/neutron 7.0.1 release.

tags: removed: liberty-backport-potential
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.