Cannot rebuild baremetal instance when vm_state is ERROR

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

Ordinarily this would be High because it was a regression, but it's been regressed so long I'm setting it to Medium.

Reviewed: https://review.opendev.org/523559
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=1819718e798fb904644391badc3beb40c181ac39
Submitter: Zuul
Branch: master

commit 1819718e798fb904644391badc3beb40c181ac39
Author: Mathieu Gagné <email address hidden>
Date: Mon Nov 27 17:33:50 2017 -0500

    Fix rebuild of baremetal instance when vm_state is ERROR

    Nova allows rebuild of instance when vm_state is ERROR. [1]

    The vm_state is restored to ACTIVE only after a successful build.
    This means rebuilding a baremetal instance using the Ironic driver
    is impossible because wait_for_active fails if vm_state=ERROR is found.

    This is a regression introduced in a previous change which added
    the ability to delete an instance in spawning state. [2]

    This present change will skip the abort installation logic
    if task_state is REBUILD_SPAWNING while preserving the previous logic.

    [1] https://bugs.launchpad.net/nova/+bug/1183946
    [2] https://bugs.launchpad.net/nova/+bug/1455000

    Change-Id: I857ad7264f1a7ef1263d8a9d4eca491d6c8dce0f
    Closes-bug: #1735009

Fix proposed to branch: stable/stein
Review: https://review.opendev.org/680869

Fix proposed to branch: stable/rocky
Review: https://review.opendev.org/680871

Fix proposed to branch: stable/queens
Review: https://review.opendev.org/680873

This issue was fixed in the openstack/nova 20.0.0.0rc1 release candidate.

Reviewed: https://review.opendev.org/680871
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=e8f418909eb0f6c319e28d6a1eac0471a0a9cee8
Submitter: Zuul
Branch: stable/rocky

commit e8f418909eb0f6c319e28d6a1eac0471a0a9cee8
Author: Mathieu Gagné <email address hidden>
Date: Mon Nov 27 17:33:50 2017 -0500

    Fix rebuild of baremetal instance when vm_state is ERROR

    Nova allows rebuild of instance when vm_state is ERROR. [1]

    The vm_state is restored to ACTIVE only after a successful build.
    This means rebuilding a baremetal instance using the Ironic driver
    is impossible because wait_for_active fails if vm_state=ERROR is found.

    This is a regression introduced in a previous change which added
    the ability to delete an instance in spawning state. [2]

    This present change will skip the abort installation logic
    if task_state is REBUILD_SPAWNING while preserving the previous logic.

    [1] https://bugs.launchpad.net/nova/+bug/1183946
    [2] https://bugs.launchpad.net/nova/+bug/1455000

    Change-Id: I857ad7264f1a7ef1263d8a9d4eca491d6c8dce0f
    Closes-bug: #1735009
    (cherry picked from commit 1819718e798fb904644391badc3beb40c181ac39)
    (cherry picked from commit c21cbf296495b0604cc995d5d17ed164ae8562c5)

This issue was fixed in the openstack/nova 19.0.3 release.

This issue was fixed in the openstack/nova 18.2.3 release.

Reviewed: https://review.opendev.org/680873
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=81056d1293ef47ae47cd5f89a7b7937dedf43c3c
Submitter: Zuul
Branch: stable/queens

commit 81056d1293ef47ae47cd5f89a7b7937dedf43c3c
Author: Mathieu Gagné <email address hidden>
Date: Mon Nov 27 17:33:50 2017 -0500

    Fix rebuild of baremetal instance when vm_state is ERROR

    Nova allows rebuild of instance when vm_state is ERROR. [1]

    The vm_state is restored to ACTIVE only after a successful build.
    This means rebuilding a baremetal instance using the Ironic driver
    is impossible because wait_for_active fails if vm_state=ERROR is found.

    This is a regression introduced in a previous change which added
    the ability to delete an instance in spawning state. [2]

    This present change will skip the abort installation logic
    if task_state is REBUILD_SPAWNING while preserving the previous logic.

    [1] https://bugs.launchpad.net/nova/+bug/1183946
    [2] https://bugs.launchpad.net/nova/+bug/1455000

    Change-Id: I857ad7264f1a7ef1263d8a9d4eca491d6c8dce0f
    Closes-bug: #1735009
    (cherry picked from commit 1819718e798fb904644391badc3beb40c181ac39)
    (cherry picked from commit c21cbf296495b0604cc995d5d17ed164ae8562c5)
    (cherry picked from commit e8f418909eb0f6c319e28d6a1eac0471a0a9cee8)

This issue was fixed in the openstack/nova 17.0.13 release.