yahoo-eng-team team mailing list archive

[OpenStack Infra <1746558@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.openstack.org/539590
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=5fe3a01b604c29b0961d19b44a873cca78891381
Submitter: Zuul
Branch:    master

commit 5fe3a01b604c29b0961d19b44a873cca78891381
Author: Dan Smith <dansmith@xxxxxxxxxx>
Date:   Wed Jan 31 08:13:36 2018 -0800

    Make service all-cells min version helper use scatter-gather
    
    This makes the Service get_minimum_version_all_cells() helper use the parallel
    scatter-gather utilities to make the determination. It also adds fault handling
    for cases where a cell fails to respond, and an option to ignore or raise in
    that situation.
    
    Closes-Bug: #1746558
    Change-Id: I62dc1033437f91bded28a9fd58759d2dbd7e55fb


** 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/1746558

Title:
  Make service all-cells min version helper use scatter-gather

Status in OpenStack Compute (nova):
  Fix Released
Status in OpenStack Compute (nova) pike series:
  Confirmed
Status in OpenStack Compute (nova) queens series:
  Confirmed

Bug description:
  Currently the "get_minimum_version_all_cells" function in service runs
  sequentially and this affects the performance in case of large
  deployments (running a lot of cells) :
  https://github.com/openstack/nova/blob/stable/pike/nova/objects/service.py#L440

  So it would be nice to use the scatter_gather_all_cells function to do
  this operation in parallel.

  Also apart from the performance scaling point of view, in case
  connection to a particular cell fails, it would be nice to have
  sentinels returned which is done by the scatter_gather_all_cells. This
  helps when a cell is down.

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