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]>
  • Loading branch information
clwluvw authored and guits committed Mar 18, 2024
1 parent fa2eb25 commit ed4b6e7
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 ed4b6e7

Please sign in to comment.