From 56ee39a7e13417203c5e1816d7a3184a07f85826 Mon Sep 17 00:00:00 2001 From: Matthew Oliver Date: Thu, 9 Jul 2015 15:19:32 +1000 Subject: [PATCH] Ring builder code clean up follow up patch This is a simple change that cleans up a NIT from Sam's 'stop moving partitions unnecessarily when overload is on' patch. Change-Id: I9d9f1cc23e2bb625d8e158f4d3f64e10973176a1 --- swift/common/ring/builder.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/swift/common/ring/builder.py b/swift/common/ring/builder.py index bf0c39dcd2..4e41f31c98 100644 --- a/swift/common/ring/builder.py +++ b/swift/common/ring/builder.py @@ -687,7 +687,7 @@ class RingBuilder(object): bigness_threshold = ( max_child_replicas / tier_replicas * tier_weight[tier]) - for child_tier in tier2children[tier]: + for child_tier in child_tiers: child_weight = tier_weight[child_tier] if child_weight == 0: # If it's got 0 weight, it's not taking any