summaryrefslogtreecommitdiff
path: root/nova/scheduler
diff options
context:
space:
mode:
authorMatt Riedemann <mriedem.os@gmail.com>2019-09-05 17:12:24 -0400
committerMatt Riedemann <mriedem.os@gmail.com>2019-09-05 17:20:49 -0400
commitd11a335ea5a705a47982105f28553e50df58b0be (patch)
treec973d88b114b72cc1b1e27dd93b5724fa6e2f25f /nova/scheduler
parentc84400f152e305150dfd939345cf21517a8af007 (diff)
downloadnova-d11a335ea5a705a47982105f28553e50df58b0be.tar.gz
Remove old comments about caching scheduler compat
The CachingScheduler driver was removed in Stein [1] but there are a couple of comments in the FilterScheduler driver code that reference conditions from when the CachingScheduler existed. This removes them to avoid confusion. [1] I1832da2190be5ef2b04953938860a56a43e8cddf Change-Id: If71a7ac7a33dbc45f417368e3e705f444a3e5bd3
Diffstat (limited to 'nova/scheduler')
-rw-r--r--nova/scheduler/filter_scheduler.py11
1 files changed, 6 insertions, 5 deletions
diff --git a/nova/scheduler/filter_scheduler.py b/nova/scheduler/filter_scheduler.py
index 672f23077e..801ae58999 100644
--- a/nova/scheduler/filter_scheduler.py
+++ b/nova/scheduler/filter_scheduler.py
@@ -173,10 +173,10 @@ class FilterScheduler(driver.Scheduler):
if (instance_uuids is None or
not self.USES_ALLOCATION_CANDIDATES or
alloc_reqs_by_rp_uuid is None):
- # We need to support the caching scheduler, which doesn't use the
- # placement API (and has USES_ALLOCATION_CANDIDATE = False) and
- # therefore we skip all the claiming logic for that scheduler
- # driver. Also, if there was a problem communicating with the
+ # We still support external scheduler drivers that don't use the
+ # placement API (and set USES_ALLOCATION_CANDIDATE = False) and
+ # therefore we skip all the claiming logic for those scheduler
+ # drivers. Also, if there was a problem communicating with the
# placement API, alloc_reqs_by_rp_uuid will be None, so we skip
# claiming in that case as well. In the case where instance_uuids
# is None, that indicates an older conductor, so we need to return
@@ -479,7 +479,8 @@ class FilterScheduler(driver.Scheduler):
"""Template method, so a subclass can implement caching."""
# NOTE(jaypipes): provider_summaries being None is treated differently
# from an empty dict. provider_summaries is None when we want to grab
- # all compute nodes, for instance when using the caching scheduler.
+ # all compute nodes, for instance when using a scheduler driver that
+ # sets USES_ALLOCATION_CANDIDATES=False.
# The provider_summaries variable will be an empty dict when the
# Placement API found no providers that match the requested
# constraints, which in turn makes compute_uuids an empty list and