commit 92fed026103b47fa2a76ea09204a4ba24c21e191
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.
Reviewed: https:/ /review. opendev. org/694389 /git.openstack. org/cgit/ openstack/ nova/commit/ ?id=92fed026103 b47fa2a76ea0920 4a4ba24c21e191
Committed: https:/
Submitter: Zuul
Branch: master
commit 92fed026103b47f a2a76ea09204a4b a24c21e191
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 I0bd63b655ad3d3 d39af8d15c781ce 0a45efc8e3a {service_ id} fail with a 409
which made DELETE /os-services/
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.
Change-Id: I70e06c607045a1 c0842f13069e51f ef438012a9c
Closes-Bug: #1852610