Cannot evacuate instances with cells v1

Bug #1552046 reported by Mathieu Gagné
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
Mathieu Gagné
Kilo
Won't Fix
Undecided
Unassigned
Liberty
Fix Released
Medium
Matt Riedemann

Bug Description

When trying to evacuate and select a destination host with cells v1, the instances stays in "REBUILD" state forever.

The command used to test evacuate is:

  nova evacuate <INSTANCE_UUID> 'api!<CELL_NAME>@<NODE_NAME>'

Changed in nova:
assignee: nobody → Mathieu Gagné (mgagne)
status: New → In Progress
Matt Riedemann (mriedem)
Changed in nova:
importance: Undecided → High
tags: added: api cells evacuate
Changed in nova:
importance: High → Medium
tags: added: kilo-backport-potential liberty-backport-potential
Revision history for this message
Matt Riedemann (mriedem) wrote :

The fix for mitaka is here: https://review.openstack.org/#/c/227440/

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

Reviewed: https://review.openstack.org/227440
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=022802997c10fc4ed56b1e1875cd7ccb16cc0688
Submitter: Jenkins
Branch: master

commit 022802997c10fc4ed56b1e1875cd7ccb16cc0688
Author: Mathieu Gagné <email address hidden>
Date: Thu Sep 24 13:43:27 2015 -0400

    Fix evacuate support with Nova cells v1

    Cells v1 does not properly support evacuate when destination node is provided.
    If a destination node is provided, evacuated instance will stay
    in 'REBUILDING' state forever.

    The evacuate method expects host to be the actual node name,
    not one with complete cell_path. Stripping the cell_path from the host
    fixes the problem.

    Closes-bug: #1552046
    Change-Id: Ib48990100ecc02325d323c8e933a859fa839a1a2

Changed in nova:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to nova (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/287830

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

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

Matt Riedemann (mriedem)
tags: removed: liberty-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/liberty)

Reviewed: https://review.openstack.org/287831
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=4e1957f084e34baac7da30780abade03a32fcd45
Submitter: Jenkins
Branch: stable/liberty

commit 4e1957f084e34baac7da30780abade03a32fcd45
Author: Mathieu Gagné <email address hidden>
Date: Thu Sep 24 13:43:27 2015 -0400

    Fix evacuate support with Nova cells v1

    Cells v1 does not properly support evacuate when destination node is provided.
    If a destination node is provided, evacuated instance will stay
    in 'REBUILDING' state forever.

    The evacuate method expects host to be the actual node name,
    not one with complete cell_path. Stripping the cell_path from the host
    fixes the problem.

    NOTE(mriedem): The test_compute_cells test is modified slightly to
    hard-code the instance uuid since the uuids sentinel code was not
    in liberty.

    Closes-bug: #1552046
    Change-Id: Ib48990100ecc02325d323c8e933a859fa839a1a2
    (cherry picked from commit 022802997c10fc4ed56b1e1875cd7ccb16cc0688)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to nova (master)

Reviewed: https://review.openstack.org/287830
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=83c1fa6c03a61580212241d566b4bc04c29fe776
Submitter: Jenkins
Branch: master

commit 83c1fa6c03a61580212241d566b4bc04c29fe776
Author: Matt Riedemann <email address hidden>
Date: Thu Mar 3 09:29:30 2016 -0500

    Enable rebuild tests in cellsv1 job

    Change 022802997c10fc4ed56b1e1875cd7ccb16cc0688 fixed evacuate/rebuild
    in the cells API so enable the rebuild-related tests we were skipping
    before.

    Related-Bug: #1445629
    Related-Bug: #1445631
    Related-Bug: #1552046

    Change-Id: I15f867d8c6ff280bc68a6abf8b0da6e970c2bb39

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.