Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PR #1998/d3edef23 backport][stable-7] Big IAM code refactor #2005

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Feb 27, 2024

This is a backport of PR #1998 as merged into main (d3edef2).

SUMMARY

Refactored code to use AnsibleIAMError and IAMErrorHandler as well as moving shared code into module_utils.iam

iam_role_info - Deprecate support for paths without leading and trailing \

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME
  • iam_access_key
  • iam_access_key_info
  • iam_group
  • iam_instance_profile
  • iam_instance_profile_info
  • iam_managed_policy
  • iam_mfa_device_info
  • iam_role
  • iam_role_info
  • iam_user
  • iam_user_info
ADDITIONAL INFORMATION

Big IAM code refactor

SUMMARY
Refactored code to use AnsibleIAMError and IAMErrorHandler as well as moving shared code into module_utils.iam
iam_role_info - Deprecate support for paths without leading and trailing \
ISSUE TYPE

Feature Pull Request

COMPONENT NAME

iam_access_key
iam_access_key_info
iam_group
iam_instance_profile
iam_instance_profile_info
iam_managed_policy
iam_mfa_device_info
iam_role
iam_role_info
iam_user
iam_user_info

ADDITIONAL INFORMATION

Reviewed-by: Helen Bailey <[email protected]>
Reviewed-by: GomathiselviS
Reviewed-by: Mark Chappell
Reviewed-by: Alina Buzachis
(cherry picked from commit d3edef2)
@patchback patchback bot mentioned this pull request Feb 27, 2024
Copy link
Contributor

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/b55a45d623744101adf2dab8687115c9

✔️ ansible-galaxy-importer SUCCESS in 5m 16s
✔️ build-ansible-collection SUCCESS in 15m 35s
✔️ ansible-test-splitter SUCCESS in 6m 11s
✔️ integration-amazon.aws-1 SUCCESS in 13m 04s
✔️ integration-amazon.aws-2 SUCCESS in 6m 15s
✔️ integration-amazon.aws-3 SUCCESS in 6m 19s
✔️ integration-amazon.aws-4 SUCCESS in 7m 18s
✔️ integration-amazon.aws-5 SUCCESS in 6m 08s
✔️ integration-amazon.aws-6 SUCCESS in 6m 56s
✔️ integration-amazon.aws-7 SUCCESS in 6m 54s
✔️ integration-amazon.aws-8 SUCCESS in 10m 33s
✔️ integration-community.aws-1 SUCCESS in 30m 00s
✔️ integration-community.aws-2 SUCCESS in 6m 47s
✔️ integration-community.aws-3 SUCCESS in 5m 39s
Skipped 33 jobs

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Feb 28, 2024
Copy link
Contributor

Build succeeded (gate pipeline).
https://ansible.softwarefactory-project.io/zuul/buildset/6bc1cee791a34c519b3600193a0f41fe

✔️ ansible-galaxy-importer SUCCESS in 5m 20s
✔️ build-ansible-collection SUCCESS in 15m 59s
✔️ ansible-test-splitter SUCCESS in 6m 22s
✔️ integration-amazon.aws-1 SUCCESS in 13m 04s
✔️ integration-amazon.aws-2 SUCCESS in 6m 23s
✔️ integration-amazon.aws-3 SUCCESS in 6m 24s
✔️ integration-amazon.aws-4 SUCCESS in 7m 38s
✔️ integration-amazon.aws-5 SUCCESS in 6m 30s
✔️ integration-amazon.aws-6 SUCCESS in 6m 24s
✔️ integration-amazon.aws-7 SUCCESS in 6m 10s
✔️ integration-amazon.aws-8 SUCCESS in 9m 58s
✔️ integration-community.aws-1 SUCCESS in 16m 50s
✔️ integration-community.aws-2 SUCCESS in 7m 27s
✔️ integration-community.aws-3 SUCCESS in 6m 35s
Skipped 33 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit b9d09e0 into stable-7 Feb 28, 2024
39 of 40 checks passed
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-7/d3edef2333ceb3aaa826c1ef870accce9848d096/pr-1998 branch February 28, 2024 08:39
Copy link

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant