Comment 33 for bug 1713783

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to nova (stable/pike)

Reviewed: https://review.openstack.org/621203
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=c5cd6c8e50201dfaa192cdfbbc64d714eeadcd95
Submitter: Zuul
Branch: stable/pike

commit c5cd6c8e50201dfaa192cdfbbc64d714eeadcd95
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: Ia0c67aeffe41e133de9dee5080be9c5ec1739aa6
    Related-Bug: #1713783
    (cherry picked from commit aec81e7d98c1ca082abd157dab5cc83b5bedfd89)