diff options
author | Matt Riedemann <mriedem.os@gmail.com> | 2019-09-05 17:12:24 -0400 |
---|---|---|
committer | Matt Riedemann <mriedem.os@gmail.com> | 2019-09-05 17:20:49 -0400 |
commit | d11a335ea5a705a47982105f28553e50df58b0be (patch) | |
tree | c973d88b114b72cc1b1e27dd93b5724fa6e2f25f /nova/scheduler | |
parent | c84400f152e305150dfd939345cf21517a8af007 (diff) | |
download | nova-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.py | 11 |
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 |