Skip to content

Commit

Permalink
Remove deprecated "facts" aliases (ansible-collections#814)
Browse files Browse the repository at this point in the history
Remove deprecated "facts" aliases

SUMMARY
Modules named "facts.py" that do not return ansible_facts were renamed to "info.py" in 2.9. Remove these aliases now that the deprecation period is over.
This PR should be included in 3.0.0 of the collection.
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
*_facts.py

Reviewed-by: Mark Chappell <None>
Reviewed-by: Jill R <None>
Reviewed-by: None <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@68aaa70
  • Loading branch information
jillr authored and abikouo committed Oct 20, 2023
1 parent c2efe19 commit f66d443
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 4 deletions.
1 change: 0 additions & 1 deletion plugins/modules/iam_role_facts.py

This file was deleted.

3 changes: 0 additions & 3 deletions plugins/modules/iam_role_info.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@
short_description: Gather information on IAM roles
description:
- Gathers information about IAM roles.
- This module was called C(iam_role_facts) before Ansible 2.9. The usage did not change.
author:
- "Will Thames (@willthames)"
options:
Expand Down Expand Up @@ -242,8 +241,6 @@ def main():
module = AnsibleAWSModule(argument_spec=argument_spec,
supports_check_mode=True,
mutually_exclusive=[['name', 'path_prefix']])
if module._name == 'iam_role_facts':
module.deprecate("The 'iam_role_facts' module has been renamed to 'iam_role_info'", date='2021-12-01', collection_name='community.aws')

client = module.client('iam', retry_decorator=AWSRetry.jittered_backoff())

Expand Down

0 comments on commit f66d443

Please sign in to comment.