OpenStack Compute (nova)

  1. Series queens
  2. Bug #1746209
  3. Comment #9

Comment 9 for bug 1746209

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

Reviewed: https://review.openstack.org/539508
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=8bbad196a7f6a6e2ea093aeee87dfde2154c9358
Submitter: Zuul
Branch: master

commit 8bbad196a7f6a6e2ea093aeee87dfde2154c9358
Author: Mark Goddard <email address hidden>
Date: Fri Jan 26 14:55:34 2018 +0000

    Include only required fields in ironic node cache

    The ironic virt driver maintains a cache of ironic nodes to avoid
    continually polling the ironic API. Code paths requiring a specific
    node use a limited set of fields, _NODE_FIELDS, when querying the
    ironic API for the node. This reduces the memory footprint required by
    the cache, and the network traffic required to populate it. However,
    in most cases the cache is populated using a detailed node list
    operation in _refresh_cache(), which includes all node fields.

    This change specifies _NODE_FIELDS in the node list operation in
    _refresh_cache().

    We also modify the unit tests to use fake node objects that are
    representative of the nodes in the cache.

    Change-Id: Id96e7e513f469b87992ddae1431cce714e91ed16
    Related-Bug: #1746209