summaryrefslogtreecommitdiff
path: root/swift/common
diff options
context:
space:
mode:
authorTim Burke <tim.burke@gmail.com>2023-02-17 13:47:53 -0800
committerTim Burke <tim.burke@gmail.com>2023-02-17 13:47:53 -0800
commit7f5576d3e6ccdd26da631e039984188bbc6a3705 (patch)
treef3b8a9faaedb029ad5ffcacde19cc85f38342214 /swift/common
parent8778b2c1328226cbcb67cd38fbb9e0771e2162fc (diff)
downloadswift-7f5576d3e6ccdd26da631e039984188bbc6a3705.tar.gz
ring-builder: require part_power>=0, give better error messages
Closes-Bug: #1697860 Change-Id: I500a86de390b24b9d08a478d695a7d62c447e779
Diffstat (limited to 'swift/common')
-rw-r--r--swift/common/ring/builder.py3
1 files changed, 3 insertions, 0 deletions
diff --git a/swift/common/ring/builder.py b/swift/common/ring/builder.py
index e64fe4089..91845070e 100644
--- a/swift/common/ring/builder.py
+++ b/swift/common/ring/builder.py
@@ -87,6 +87,9 @@ class RingBuilder(object):
if part_power > 32:
raise ValueError("part_power must be at most 32 (was %d)"
% (part_power,))
+ if part_power < 0:
+ raise ValueError("part_power must be at least 0 (was %d)"
+ % (part_power,))
if replicas < 1:
raise ValueError("replicas must be at least 1 (was %.6f)"
% (replicas,))