commit b30cece36ea143a97544799fb807644a51fd1c60
Author: Jason Anderson <email address hidden>
Date: Fri Oct 11 16:51:28 2019 -0500
Fix issue moving hosts back to freepool
If an aggregate fails to be created for some reason, for example because
the requested node is not in the freepool, some cleanup occurs. Part of
the cleanup is ensuring the target node is moved _back_ to the freepool.
This should theoretically "heal" cases where the node wasn't in the
freepool but also wasn't in some lease aggregate.
However, this call to Nova's API refers to the freepool aggregate by its
name, which is not supported here: the ID must be used[1]. This caused
this operation to fail and raise a NotFound, confusingly (because Nova
couldn't find an aggregate with ID='freepool' for example.)
Reviewed: https:/ /review. opendev. org/689541 /git.openstack. org/cgit/ openstack/ blazar/ commit/ ?id=b30cece36ea 143a97544799fb8 07644a51fd1c60
Committed: https:/
Submitter: Zuul
Branch: stable/train
commit b30cece36ea143a 97544799fb80764 4a51fd1c60
Author: Jason Anderson <email address hidden>
Date: Fri Oct 11 16:51:28 2019 -0500
Fix issue moving hosts back to freepool
If an aggregate fails to be created for some reason, for example because
the requested node is not in the freepool, some cleanup occurs. Part of
the cleanup is ensuring the target node is moved _back_ to the freepool.
This should theoretically "heal" cases where the node wasn't in the
freepool but also wasn't in some lease aggregate.
However, this call to Nova's API refers to the freepool aggregate by its
name, which is not supported here: the ID must be used[1]. This caused
this operation to fail and raise a NotFound, confusingly (because Nova
couldn't find an aggregate with ID='freepool' for example.)
[1]: /docs.openstack .org/api- ref/compute/ ?expanded= add-host- detail# add-host
https:/
Closes-Bug: #1847821 8617131f0de42be cb3dc2bc415 92d0b72a6dbaa1f 05864450e0)
Change-Id: I7af4d407b18357
(cherry picked from commit 0346b02ccea75b9