Skip to content

Commit

Permalink
Revert breaking change - iam_role return values (ansible-collections#…
Browse files Browse the repository at this point in the history
…1068)

Revert breaking change - iam_role return values

SUMMARY
This hasn't been release yet, so a changelog isn't needed.
While I'm generally good with cleaning up the output values here, this needs to be done as a separate breaking change, and must not be backported to stable-3.
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
iam_role
ADDITIONAL INFORMATION
Breaking change silently introduced by ansible-collections#1054

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Joseph Torcasso <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@8d80e9a
  • Loading branch information
tremble authored and alinabuzachis committed Oct 6, 2023
1 parent d7c287e commit 83e1b4f
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion plugins/modules/iam_role.py
Original file line number Diff line number Diff line change
Expand Up @@ -506,7 +506,8 @@ def create_or_update_role(module, client):
role['AttachedPolicies'] = get_attached_policy_list(module, client, role_name)
role['tags'] = get_role_tags(module, client)

module.exit_json(changed=changed, iam_role=camel_dict_to_snake_dict(role, ignore_list=['tags']))
camel_role = camel_dict_to_snake_dict(role, ignore_list=['tags'])
module.exit_json(changed=changed, iam_role=camel_role, **camel_role)


def create_instance_profiles(module, client, role_name, path):
Expand Down

0 comments on commit 83e1b4f

Please sign in to comment.