Comment 9 for bug 1816360

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

Reviewed: https://review.openstack.org/641359
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=bc2c3359b32f452cd772e510d4c139a8e4594900
Submitter: Zuul
Branch: stable/queens

commit bc2c3359b32f452cd772e510d4c139a8e4594900
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
    (cherry picked from commit 84533f5eb3c5b4ab7598d7c278b53524acc1c6e0)
    (cherry picked from commit 31b74bfa4063d68d0f5ea9e883cad8cbcb70ab09)