← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1819430] Re: unnecessary placement call to fill group mapping during boot

 

Reviewed:  https://review.openstack.org/638711
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=1a89c1d73b8230e437ed0e15656a2606a94926f8
Submitter: Zuul
Branch:    master

commit 1a89c1d73b8230e437ed0e15656a2606a94926f8
Author: Balazs Gibizer <balazs.gibizer@xxxxxxxxxxxx>
Date:   Fri Feb 22 11:34:31 2019 +0100

    Use Selection object to fill request group mapping
    
    To fill the request group - resource provider mapping nova needs to look
    at the allocation that is made in placement as well as the traits of
    the used providers. So far these were explicit placement calls. However
    it turns out that the Selection object already contains the allocation
    data so one of the extra placement calls can be removed.
    
    This patch replaces the GET /allocations/<uuid> placement call from
    conductor build instance codepath with the usage of the
    Selection.allocation_request json blob.
    
    Closes-Bug: #1819430
    
    Change-Id: Iecbee518444bd282ce5f6fd019db41a322f76a83


** Changed in: nova
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1819430

Title:
  unnecessary placement call to fill group mapping during boot

Status in OpenStack Compute (nova):
  Fix Released

Bug description:
  To fill the request group - resource provider mapping nova needs to
  look at the allocation that is made in placement as well as the traits
  of the used providers. These are explicit placement calls today.
  However it turns out that the Selection object already contains the
  allocation data so one of the extra placement call can be removed.
  This change can lead to a bit better performance.

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1819430/+subscriptions


References