Skip to content

Commit

Permalink
ceph_config: compare current vals in low
Browse files Browse the repository at this point in the history
Check the current value and asked both in low case as the asked one might be in capital as it should be.

Signed-off-by: Seena Fallah <[email protected]>
(cherry picked from commit ed4b6e7)
  • Loading branch information
clwluvw authored and guits committed Mar 19, 2024
1 parent cdc9e6b commit 127dab7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion library/ceph_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,7 @@ def main() -> None:
current_value = get_current_value(who, option, config_dump)

if action == 'set':
if value.lower() == current_value:
if current_value and value.lower() == current_value.lower():
out = 'who={} option={} value={} already set. Skipping.'.format(who, option, value)
else:
rc, cmd, out, err = set_option(module, who, option, value, container_image=container_image)
Expand Down

0 comments on commit 127dab7

Please sign in to comment.