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

ARN: Add Support for the aws/aws-managed 'account' special cases #760

Merged

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Mar 30, 2022

SUMMARY

AWS has a couple of special cases for accounts in ARNs where they're named instead of just numbers.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/module_utils/arn.py

ADDITIONAL INFORMATION

This code isn't part of a release yet, no need for a changelog

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request module_utils module_utils needs_triage plugins plugin (any type) tests tests labels Mar 30, 2022
@softwarefactory-project-zuul

This comment was marked as resolved.

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@tremble
Copy link
Contributor Author

tremble commented Mar 31, 2022

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@ansibullbot
Copy link

@ansibullbot ansibullbot added the module module label Mar 31, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Apr 5, 2022
@goneri goneri closed this Apr 5, 2022
@goneri goneri reopened this Apr 5, 2022
@goneri
Copy link
Member

goneri commented Apr 5, 2022

regate

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit eb8c4a5 into ansible-collections:main Apr 5, 2022
@tremble tremble deleted the arn/aws_special branch April 20, 2022 07:28
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…ions

Docs - cleanup Ansible Version related docs

SUMMARY
We had some free-form docs entries referring to old versions of Ansible which pre-date the collection.  In line with removing the version_added data from prior to the collection's existence, remove these.
Additionally we had a mix of Ansible version (in free-form docs) and Date deprecations (in module.deprecate), move these all over to dates.
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
plugins/modules/aws_kms.py
plugins/modules/cloudfront_info.py
plugins/modules/ec2_lc.py
plugins/modules/ecs_taskdefinition.py
plugins/modules/elb_network_lb.py
plugins/modules/iam_policy.py
plugins/modules/iam_role.py
plugins/modules/route53.py
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
Reviewed-by: None <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
…ions

Docs - cleanup Ansible Version related docs

SUMMARY
We had some free-form docs entries referring to old versions of Ansible which pre-date the collection.  In line with removing the version_added data from prior to the collection's existence, remove these.
Additionally we had a mix of Ansible version (in free-form docs) and Date deprecations (in module.deprecate), move these all over to dates.
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
plugins/modules/aws_kms.py
plugins/modules/cloudfront_info.py
plugins/modules/ec2_lc.py
plugins/modules/ecs_taskdefinition.py
plugins/modules/elb_network_lb.py
plugins/modules/iam_policy.py
plugins/modules/iam_role.py
plugins/modules/route53.py
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
Reviewed-by: None <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review feature This issue/PR relates to a feature request mergeit Merge the PR (SoftwareFactory) module_utils module_utils module module needs_triage plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants