The docstring on this method no longer reflects reality,
and hasn't since we started filtering on migration records
starting back in Liberty. This change updates the docstring
for what the method actually does now, and also includes a
comment about why we include 'accepted' but not yet complete
migrations when looking for things to remove from this source
node.
Change-Id: Ia0c67aeffe41e133de9dee5080be9c5ec1739aa6
Related-Bug: #1713783
(cherry picked from commit aec81e7d98c1ca082abd157dab5cc83b5bedfd89)
Reviewed: https:/ /review. openstack. org/621203 /git.openstack. org/cgit/ openstack/ nova/commit/ ?id=c5cd6c8e502 01dfaa192cdfbbc 64d714eeadcd95
Committed: https:/
Submitter: Zuul
Branch: stable/pike
commit c5cd6c8e50201df aa192cdfbbc64d7 14eeadcd95
Author: Matt Riedemann <email address hidden>
Date: Fri Sep 1 13:56:38 2017 -0400
Update docs for _destroy_ evacuated_ instances
The docstring on this method no longer reflects reality,
and hasn't since we started filtering on migration records
starting back in Liberty. This change updates the docstring
for what the method actually does now, and also includes a
comment about why we include 'accepted' but not yet complete
migrations when looking for things to remove from this source
node.
Change-Id: Ia0c67aeffe41e1 33de9dee5080be9 c5ec1739aa6 82abd157dab5cc8 3b5bedfd89)
Related-Bug: #1713783
(cherry picked from commit aec81e7d98c1ca0