Ironic: get_available_resource doesn't report numa_topology

Bug #1483486 reported by Eli Qiao
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Undecided
Eli Qiao

Bug Description

in update_available_resource of RT we have

        # TODO(berrange): remove this once all virt drivers are updated
        # to report topology
        if "numa_topology" not in resources:
            resources["numa_topology"] = None

by searching codes, all other hypervisor report numa_topology in get_available_resource except ironic driver.

this can be improved.

Tags: ironic
Eli Qiao (taget-9)
Changed in nova:
assignee: nobody → Eli Qiao (taget-9)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.openstack.org/214425

Changed in nova:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

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

commit 9afb264151998baaa921a034219da3b8e17347e0
Author: Eli Qiao <email address hidden>
Date: Wed Aug 12 16:25:53 2015 +0800

    Ironic: Add numa_topology to get_available_resource return values

    All other hypervisor drivers return numa_topology except ironic driver in
    get_available_resource, we need to align them to avoid extra judgements
    when doing update_available_resource in resource tracker.

    Closes-Bug: #1483486
    Change-Id: Ib985929265bc2af22f2416ce741672c1d64d88fd

Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → liberty-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: liberty-3 → 12.0.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers