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 #2068/970c3032 backport][stable-7] iam_managed_policy - fix ParamValidationError during policy deletion #2071

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Apr 30, 2024

This is a backport of PR #2068 as merged into main (970c303).

SUMMARY

fixes: #2067
Introduced by #1998

A copy and paste mistake in #1998 resulted in ParamValidationErrors being triggered when deleting a managed policy which is still attached to a role or user.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

iam_managed_policy
plugins/module_utils/iam.py

ADDITIONAL INFORMATION

…mValidationError during policy deletion (#2068)

iam_managed_policy - fix ParamValidationError during policy deletion

SUMMARY
fixes: #2067
Introduced by #1998
A copy and paste mistake in #1998 resulted in ParamValidationErrors being triggered when deleting a managed policy which is still attached to a role or user.
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
iam_managed_policy
plugins/module_utils/iam.py
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis
(cherry picked from commit 970c303)
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Apr 30, 2024
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 5m 32s
✔️ build-ansible-collection SUCCESS in 15m 15s
✔️ ansible-test-splitter SUCCESS in 6m 15s
✔️ integration-amazon.aws-1 SUCCESS in 13m 19s
✔️ integration-amazon.aws-2 SUCCESS in 6m 27s
✔️ integration-amazon.aws-3 SUCCESS in 8m 36s
✔️ integration-amazon.aws-4 SUCCESS in 5m 11s
✔️ integration-community.aws-1 SUCCESS in 24m 51s
✔️ integration-community.aws-2 SUCCESS in 7m 57s
✔️ integration-community.aws-3 SUCCESS in 5m 44s
Skipped 37 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 2db7202 into stable-7 Apr 30, 2024
39 of 40 checks passed
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-7/970c30329167632e94ecce321cec5a9112757075/pr-2068 branch April 30, 2024 13:47
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