Comment 16 for bug 1799727

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

Reviewed: https://review.openstack.org/614563
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=57ca9288ce33550b73deee22ca902cc35f2b4819
Submitter: Zuul
Branch: stable/rocky

commit 57ca9288ce33550b73deee22ca902cc35f2b4819
Author: Matt Riedemann <email address hidden>
Date: Wed Oct 24 13:08:33 2018 -0400

    Add functional recreate test for bug 1799727

    This adds a functional test which recreates the
    bug where config-driven reserved and allocation ratio
    overrides are not being reflected in resource provider
    inventory once initially set.

    The reserved and allocation_ratio values set in the
    FakeDriver.update_provider_tree method, added in change
    I69d760aaf931d46f011cfd229b88f400837662e8, are removed
    here otherwise they hard-code the values which get sent
    to placement and ResourceTracker._normalize_inventory_from_cn_obj
    won't update the reserved / ratios based on config. The
    fake virt driver shouldn't really need to hard-code these
    values since the RT will provide those based on config.

    Conflicts:
          nova/virt/fake.py

    NTOE(mriedem): The conflict is due to not having change
    I69d760aaf931d46f011cfd229b88f400837662e8 in Rocky. Rather
    than backport that change, this backport simply adds the
    update_provider_tree method to the fake virt driver. The
    other noticeable difference is the "allocations" kwarg is
    ommitted since that was added in Stein with change
    Ic062446e5c620c89aec3065b34bcdc6bf5966275.

    Change-Id: Ie66d6f4c83a7d6fc64a64dbd752e427cee1356d0
    Related-Bug: #1799727
    (cherry picked from commit 45f36cebabdbaad10e69c1a99f5ad4f091885f82)