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 <[email protected]> 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 -- Mailing list: https://launchpad.net/~yahoo-eng-team Post to : [email protected] Unsubscribe : https://launchpad.net/~yahoo-eng-team More help : https://help.launchpad.net/ListHelp
