aws_kms_info: Correct deprecation of keys_attr (ansible-collections#851)

aws_kms_info: Correct deprecation of keys_attr SUMMARY fixup derecation of keys_attr. ISSUE TYPE Feature Pull Request COMPONENT NAME aws_kms_info ADDITIONAL INFORMATION ansible-collections#838 Reviewed-by: Markus Bergholz <[email protected]> Reviewed-by: Jill R <None> Reviewed-by: None <None>
abikouo · Jan 7, 2022 · c56dbee · c56dbee
1 parent cd9e061
commit c56dbee
Showing 1 changed file with 9 additions and 12 deletions.
diff --git a/aws_kms_info.py b/aws_kms_info.py
@@ -47,14 +47,11 @@
     type: bool
   keys_attr:
     description:
-      - Whether to return the results in the C(keys) attribute as well as the
-        C(kms_keys) attribute.
-      - Returning the C(keys) attribute conflicts with the builtin keys()
-        method on dictionaries and as such has been deprecated.
-      - After version C(3.0.0) this parameter will do nothing, and after
-        version C(4.0.0) this parameter will be removed.
+      - Returning the C(keys) attribute conflicted with the builtin keys()
+        method on dictionaries and as such was deprecated.
+      - This parameter now does nothing, and after version C(4.0.0) this
+        parameter will be removed.
     type: bool
-    default: True
     version_added: 2.0.0
 extends_documentation_fragment:
 - amazon.aws.aws
@@ -451,7 +448,7 @@ def main():
         key_id=dict(aliases=['key_arn']),
         filters=dict(type='dict'),
         pending_deletion=dict(type='bool', default=False),
-        keys_attr=dict(type='bool', default=True),
+        keys_attr=dict(type='bool'),
     )
 
     module = AnsibleAWSModule(argument_spec=argument_spec,
@@ -468,11 +465,11 @@ def main():
     ret_params = dict(kms_keys=filtered_keys)
 
     # We originally returned "keys"
-    if module.params['keys_attr']:
+    if module.params.get('keys_attr') is not None:
         module.deprecate("Returning results in the 'keys' attribute conflicts with the builtin keys() method on "
-                         "dicts and as such is deprecated and is now ignored. Please use the kms_keys attribute. This warning can be "
-                         "silenced by setting keys_attr to False.",
-                         version='3.0.0', collection_name='community.aws')
+                         "dicts and as such was removed in version 3.0.0.  Please use the kms_keys attribute. "
+                         "This parameter is now ignored and will be removed in version 4.0.0.",
+                         version='4.0.0', collection_name='community.aws')
     module.exit_json(**ret_params)