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

Refactor iam/boto3 "normalize" functions #2001

Closed
1 task done
tremble opened this issue Feb 27, 2024 · 1 comment · Fixed by #2006
Closed
1 task done

Refactor iam/boto3 "normalize" functions #2001

tremble opened this issue Feb 27, 2024 · 1 comment · Fixed by #2006
Assignees
Milestone

Comments

@tremble
Copy link
Contributor

tremble commented Feb 27, 2024

Summary

The vast bulk of the boto3 -> ansible normalization is common code it may be good to refactor this to reduce C&P

Issue Type

Feature Idea

Component Name

plugins/module_utils/iam.py

Additional Information

No response

Code of Conduct

  • I agree to follow the Ansible Code of Conduct
@tremble tremble added this to the 8.0.0 milestone Feb 27, 2024
@tremble tremble self-assigned this Feb 27, 2024
@GomathiselviS
Copy link
Collaborator

@tremble We are currently exploring options for refactoring our codebase, and to track these options, we've created a project board. I will add this issue to the board.

This was referenced Feb 27, 2024
softwarefactory-project-zuul bot pushed a commit that referenced this issue Mar 25, 2024
Normalize refactor

SUMMARY
fixes: #2001
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/module_utils/iam.py
plugins/module_utils/transformation.py
ADDITIONAL INFORMATION
Note: also tinkers a little with Type Hinting, not sure of the value
TODO:

 Initial transform code
 Unit tests for transform code
 initial iam updates
 Basic unit tests for iam updates
 changelog

Reviewed-by: Alina Buzachis
Reviewed-by: Bikouo Aubin
patchback bot pushed a commit that referenced this issue Mar 26, 2024
Normalize refactor

SUMMARY
fixes: #2001
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/module_utils/iam.py
plugins/module_utils/transformation.py
ADDITIONAL INFORMATION
Note: also tinkers a little with Type Hinting, not sure of the value
TODO:

 Initial transform code
 Unit tests for transform code
 initial iam updates
 Basic unit tests for iam updates
 changelog

Reviewed-by: Alina Buzachis
Reviewed-by: Bikouo Aubin
(cherry picked from commit 3974a82)
softwarefactory-project-zuul bot pushed a commit that referenced this issue Apr 1, 2024
[PR #2006/3974a823 backport][stable-7] Normalize refactor

This is a backport of PR #2006 as merged into main (3974a82).
SUMMARY
fixes: #2001
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/module_utils/iam.py
plugins/module_utils/transformation.py
ADDITIONAL INFORMATION
Note: also tinkers a little with Type Hinting, not sure of the value
TODO:

 Initial transform code
 Unit tests for transform code
 initial iam updates
 Basic unit tests for iam updates
 changelog

Reviewed-by: Helen Bailey <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging a pull request may close this issue.

3 participants