openstack/placement
Code Issues Proposed changes

Merge "Enhance debug logging in allocation candidate handling"

Browse Source
This commit is contained in:
Zuul
2019-05-20 09:35:40 +00:00
committed by Gerrit Code Review
parent 43c859b421 73b29cd6e4
commit 0f1a5dddf8
1 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
placement/objects/allocation_candidate.py
View File

@@ -252,6 +252,9 @@ class AllocationCandidates(object):
continue continue
kept_summary_objs.append(summary) kept_summary_objs.append(summary)
summary_objs = kept_summary_objs summary_objs = kept_summary_objs
LOG.debug('Limiting results yields %d allocation requests and '
'%d provider summaries', len(alloc_request_objs),
len(summary_objs))
elif context.config.placement.randomize_allocation_candidates: elif context.config.placement.randomize_allocation_candidates:
random.shuffle(alloc_request_objs) random.shuffle(alloc_request_objs)
@@ -937,6 +940,8 @@ def _merge_candidates(candidates, group_policy=None):
psums = [psum for psum in all_psums if psums = [psum for psum in all_psums if
psum.resource_provider.root_provider_uuid in tree_uuids] psum.resource_provider.root_provider_uuid in tree_uuids]
LOG.debug('Merging candidates yields %d allocation requests and %d '
'provider summaries', len(areqs), len(psums))
return list(areqs), psums return list(areqs), psums
@@ -1022,4 +1027,7 @@ def _exclude_nested_providers(allocation_requests, provider_summaries):
if ps.resource_provider.uuid not in all_rp_uuids: if ps.resource_provider.uuid not in all_rp_uuids:
provider_summaries.remove(ps) provider_summaries.remove(ps)
LOG.debug('Excluding nested providers yields %d allocation requests and '
'%d provider summaries', len(allocation_requests),
len(provider_summaries))
return allocation_requests, provider_summaries return allocation_requests, provider_summaries