diff --git a/plugins/module_utils/ansible_freeipa_module.py b/plugins/module_utils/ansible_freeipa_module.py index 3dffd44a99a8ec7e586097112850933ba46d2e1d..cd0a5b08aa221985ad76f75d689c529bfcfa2c8d 100644 --- a/plugins/module_utils/ansible_freeipa_module.py +++ b/plugins/module_utils/ansible_freeipa_module.py @@ -271,9 +271,9 @@ else: '%Y-%m-%d %H:%MZ', # non-ISO 8601, minute precision ] - for date_format in accepted_date_formats: + for _date_format in accepted_date_formats: try: - return datetime.strptime(value, date_format) + return datetime.strptime(value, _date_format) except ValueError: pass raise ValueError("Invalid date '%s'" % value) diff --git a/plugins/modules/ipaconfig.py b/plugins/modules/ipaconfig.py index 5e4a1bae01be765db9c4d0f679a500cd1b6984c9..b557baf5b5b56f190409c41a02294fae9309ec7e 100644 --- a/plugins/modules/ipaconfig.py +++ b/plugins/modules/ipaconfig.py @@ -399,11 +399,11 @@ def main(): ("ipasearchrecordslimit", -1, 2147483647), ("ipapwdexpadvnotify", 0, 2147483647), ] - for arg, min, max in args_with_limits: - if arg in params and (params[arg] > max or params[arg] < min): + for arg, minimum, maximum in args_with_limits: + if arg in params and (params[arg] > maximum or params[arg] < minimum): ansible_module.fail_json( msg="Argument '%s' must be between %d and %d." - % (arg, min, max)) + % (arg, minimum, maximum)) changed = False exit_args = {} diff --git a/plugins/modules/ipadnsforwardzone.py b/plugins/modules/ipadnsforwardzone.py index 5df7a1e89026966fda7443deb62efc6f081f5c95..1c2a22e53d84e9a0f4e4b677222a1af95e2039cb 100644 --- a/plugins/modules/ipadnsforwardzone.py +++ b/plugins/modules/ipadnsforwardzone.py @@ -386,8 +386,8 @@ def main(): **exit_args) # Execute commands - for name, command, args in commands: - api_command(ansible_module, command, name, args) + for _name, command, args in commands: + api_command(ansible_module, command, _name, args) changed = True except Exception as e: