Comment 28 for bug 1852610

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

commit 30a635068512be558acf0f9c83185dc1aaad560f
Author: Matt Riedemann <email address hidden>
Date: Thu Nov 14 14:19:26 2019 -0500

    Block deleting compute services with in-progress migrations

    This builds on I0bd63b655ad3d3d39af8d15c781ce0a45efc8e3a
    which made DELETE /os-services/{service_id} fail with a 409
    response if the host has instances on it. This change checks
    for in-progress migrations involving the nodes on the host,
    either as the source or destination nodes, and returns a 409
    error response if any are found.

    Failling to do this can lead to orphaned resource providers
    in placement and also failing to properly confirm or revert
    a pending resize or cold migration.

    A release note is included for the (justified) behavior
    change in the API. A new microversion should not be required
    for this since admins should not have to opt out of broken
    behavior.

    Conflicts:
          nova/tests/functional/integrated_helpers.py

    NOTE(mriedem): The conflict is due to not having change
    Ie991d4b53e9bb5e7ec26da99219178ab7695abf6 in Rocky.

    Change-Id: I70e06c607045a1c0842f13069e51fef438012a9c
    Closes-Bug: #1852610
    (cherry picked from commit 92fed026103b47fa2a76ea09204a4ba24c21e191)
    (cherry picked from commit a9650b3cbfc674e283964090fb64ac6297be5b78)
    (cherry picked from commit a0290858b717b4cefd0d6fc17acc2b143ab12ac4)