Comment 4 for bug 1816360

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

Reviewed: https://review.openstack.org/641143
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=84533f5eb3c5b4ab7598d7c278b53524acc1c6e0
Submitter: Zuul
Branch: master

commit 84533f5eb3c5b4ab7598d7c278b53524acc1c6e0
Author: Matt Riedemann <email address hidden>
Date: Tue Mar 5 17:16:23 2019 -0500

    Fix WeighedHost logging regression

    Change I8666e0af3f057314f6b06939a108411b8a88d64b in Pike
    refactored some code in the FilterScheduler which accidentally
    changed how the list of weighed hosts are logged, which caused
    the wrapped HostState objects to be logged rather than the
    WeighedHost objects, which contain the actual "weight" attribute
    which is useful for debugging weigher configuration and
    scheduling decisions.

    This fixes the regression by logging the weighed hosts before
    stripping off the WeighedHost wrapper and adds a simple wrinkle
    to an existing test to assert we are logging the correct object.

    Change-Id: I528794b4b6f0007efc1238ad28dc402456664f86
    Closes-Bug: #1816360