Comment 6 for bug 1405359

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/156930
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=c206d162febe7f12d84a072bda8c33fd408a343b
Submitter: Jenkins
Branch: master

commit c206d162febe7f12d84a072bda8c33fd408a343b
Author: Nikola Dipanov <email address hidden>
Date: Wed Feb 18 10:29:37 2015 +0100

    Set instance NUMA topology on HostState

    NUMATopologyFilter will try to fit an instance onto every host
    (represented by a HostState instance) so assigning the resulting
    instance topology to the instance dict really makes no sense, as we end
    up with only the last calculated topology from all the filter runs.

    This in turn makes consume_from_instance not work as expected, as it
    will consume NUMA topology calculated from the last host the filter was
    run on, not the host that was chosen by the scheduler.

    This patch stashes the calculated NUMA topology onto the HostState
    instance passed to the filter, that will be used in
    consume_from_instance, and makes sure that is what gets used for
    updating the usage.

    Change-Id: Ifacccadf73dc114e50f46b8e6087ffb2b2fc9d6b
    Closes-Bug: #1405359