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 mergify[bot] committed Mar 18, 2024
1 parent ac3081e commit e587125
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 @@ -165,7 +165,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 e587125

Please sign in to comment.