openstack/swift
Code Issues Proposed changes

Add Docstrings to validate_replicas_by_tier

Browse Source 
New common functions should have Docstrings

Change-Id: Icbb3cdf38509fd6d034cbb2271786559780a7b68
This commit is contained in:
vxlinux 2018-01-19 16:54:26 +08:00 committed by yanwei
parent e46739fffa
commit 12f8745349
1 changed files with 8 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
swift/common/ring/utils.py
View File

@ -648,6 +648,14 @@ def dispersion_report(builder, search_filter=None,
def validate_replicas_by_tier(replicas, replicas_by_tier):
"""
Validate the sum of the replicas at each tier.
The sum of the replicas at each tier should be less than or very close to
the upper limit indicated by replicas
:param replicas: float,the upper limit of replicas
:param replicas_by_tier: defaultdict,the replicas by tier
"""
tiers = ['cluster', 'regions', 'zones', 'servers', 'devices']
for i, tier_name in enumerate(tiers):
replicas_at_tier = sum(replicas_by_tier[t] for t in