diff options
author | Tim Burke <tim.burke@gmail.com> | 2023-02-17 13:47:53 -0800 |
---|---|---|
committer | Tim Burke <tim.burke@gmail.com> | 2023-02-17 13:47:53 -0800 |
commit | 7f5576d3e6ccdd26da631e039984188bbc6a3705 (patch) | |
tree | f3b8a9faaedb029ad5ffcacde19cc85f38342214 /swift/common | |
parent | 8778b2c1328226cbcb67cd38fbb9e0771e2162fc (diff) | |
download | swift-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.py | 3 |
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,)) |