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

Use TagSpecifications parameter when creating EC2 resources #1876

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Nov 23, 2023

SUMMARY

For the last couple of years Amazon's supported tagging EC2 resources as part of the creation actions. Switch the last amazon.aws EC2 modules over to using TagSpecifications during creation to support folks using Tagging requirements as part of their IAM/SCP policies

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

ec2_vpc_route_table
ec2_vpc_igw
ec2_vpc_subnet
ec2_eip

ADDITIONAL INFORMATION

fixes: #1843

@tremble tremble changed the title Use ResourceTags parameter when creating EC2 resources Use TagSpecifications parameter when creating EC2 resources Nov 23, 2023
@tremble tremble added the backport-7 PR should be backported to the stable-7 branch label Nov 23, 2023

This comment was marked as resolved.

Copy link

github-actions bot commented Nov 24, 2023

Docs Build 📝

Thank you for contribution!✨

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

This comment was marked as resolved.

This comment was marked as resolved.

ansible_dict_to_boto3_tag_list and compare_aws_tags
…keys

with "tag:" in preparation for them being used as part of a filter
@tremble tremble marked this pull request as ready for review November 27, 2023 14:50
Copy link
Contributor

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/c0cb0631474f4696b16a9dd76cdb4df4

✔️ ansible-galaxy-importer SUCCESS in 5m 16s
✔️ build-ansible-collection SUCCESS in 15m 14s
✔️ ansible-test-splitter SUCCESS in 6m 30s
✔️ integration-amazon.aws-1 SUCCESS in 21m 53s
✔️ integration-amazon.aws-2 SUCCESS in 23m 56s
✔️ integration-amazon.aws-3 SUCCESS in 34m 11s
✔️ integration-amazon.aws-4 SUCCESS in 31m 22s
✔️ integration-amazon.aws-5 SUCCESS in 48m 26s
✔️ integration-amazon.aws-6 SUCCESS in 16m 49s
✔️ integration-amazon.aws-7 SUCCESS in 41m 30s
✔️ integration-amazon.aws-8 SUCCESS in 40m 02s
✔️ integration-amazon.aws-9 SUCCESS in 34m 08s
✔️ integration-amazon.aws-10 SUCCESS in 30m 42s
✔️ integration-amazon.aws-11 SUCCESS in 22m 41s
✔️ integration-amazon.aws-12 SUCCESS in 19m 41s
✔️ integration-amazon.aws-13 SUCCESS in 23m 51s
✔️ integration-amazon.aws-14 SUCCESS in 36m 37s
✔️ integration-amazon.aws-15 SUCCESS in 25m 29s
✔️ integration-amazon.aws-16 SUCCESS in 47m 17s
✔️ integration-amazon.aws-17 SUCCESS in 48m 08s
✔️ integration-amazon.aws-18 SUCCESS in 31m 45s
✔️ integration-amazon.aws-19 SUCCESS in 15m 36s
✔️ integration-amazon.aws-20 SUCCESS in 36m 37s
✔️ integration-amazon.aws-21 SUCCESS in 39m 33s
✔️ integration-amazon.aws-22 SUCCESS in 26m 52s
✔️ integration-community.aws-1 SUCCESS in 22m 54s
✔️ integration-community.aws-2 SUCCESS in 52m 48s
✔️ integration-community.aws-3 SUCCESS in 58m 54s
✔️ integration-community.aws-4 SUCCESS in 7m 30s
✔️ integration-community.aws-5 SUCCESS in 22m 47s
✔️ integration-community.aws-6 SUCCESS in 28m 58s
✔️ integration-community.aws-7 SUCCESS in 16m 25s
✔️ integration-community.aws-8 SUCCESS in 21m 30s
✔️ integration-community.aws-9 SUCCESS in 10m 29s
✔️ integration-community.aws-10 SUCCESS in 38m 14s
✔️ integration-community.aws-11 SUCCESS in 20m 09s
✔️ integration-community.aws-12 SUCCESS in 28m 30s
integration-community.aws-13 RETRY_LIMIT in 1m 49s
✔️ integration-community.aws-14 SUCCESS in 12m 11s
✔️ integration-community.aws-15 SUCCESS in 11m 41s
Skipped 7 jobs

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Nov 28, 2023

This comment was marked as outdated.

@tremble
Copy link
Contributor Author

tremble commented Nov 28, 2023

regate

Copy link
Contributor

Build failed (gate pipeline). For information on how to proceed, see
http://docs.openstack.org/infra/manual/developers.html#automated-testing

https://ansible.softwarefactory-project.io/zuul/buildset/ee04053a20374af68a99592351bf168f

✔️ ansible-galaxy-importer SUCCESS in 3m 42s
✔️ build-ansible-collection SUCCESS in 14m 45s
✔️ ansible-test-splitter SUCCESS in 5m 43s
✔️ integration-amazon.aws-1 SUCCESS in 23m 28s
✔️ integration-amazon.aws-2 SUCCESS in 22m 42s
✔️ integration-amazon.aws-3 SUCCESS in 30m 30s
✔️ integration-amazon.aws-4 SUCCESS in 33m 17s
integration-amazon.aws-5 TIMED_OUT in 1h 01m 05s
✔️ integration-amazon.aws-6 SUCCESS in 17m 59s
✔️ integration-amazon.aws-7 SUCCESS in 45m 40s
✔️ integration-amazon.aws-8 SUCCESS in 44m 27s
✔️ integration-amazon.aws-9 SUCCESS in 28m 00s
✔️ integration-amazon.aws-10 SUCCESS in 27m 11s
✔️ integration-amazon.aws-11 SUCCESS in 21m 07s
✔️ integration-amazon.aws-12 SUCCESS in 22m 06s
✔️ integration-amazon.aws-13 SUCCESS in 23m 04s
✔️ integration-amazon.aws-14 SUCCESS in 33m 19s
✔️ integration-amazon.aws-15 SUCCESS in 23m 58s
✔️ integration-amazon.aws-16 SUCCESS in 49m 33s
✔️ integration-amazon.aws-17 SUCCESS in 44m 35s
✔️ integration-amazon.aws-18 SUCCESS in 30m 28s
✔️ integration-amazon.aws-19 SUCCESS in 17m 43s
✔️ integration-amazon.aws-20 SUCCESS in 32m 19s
✔️ integration-amazon.aws-21 SUCCESS in 45m 42s
✔️ integration-amazon.aws-22 SUCCESS in 28m 23s
✔️ integration-community.aws-1 SUCCESS in 21m 32s
✔️ integration-community.aws-2 SUCCESS in 47m 18s
✔️ integration-community.aws-3 SUCCESS in 58m 15s
✔️ integration-community.aws-4 SUCCESS in 7m 24s
✔️ integration-community.aws-5 SUCCESS in 23m 29s
✔️ integration-community.aws-6 SUCCESS in 26m 09s
✔️ integration-community.aws-7 SUCCESS in 18m 18s
✔️ integration-community.aws-8 SUCCESS in 12m 34s
✔️ integration-community.aws-9 SUCCESS in 12m 20s
✔️ integration-community.aws-10 SUCCESS in 37m 43s
✔️ integration-community.aws-11 SUCCESS in 18m 34s
✔️ integration-community.aws-12 SUCCESS in 27m 49s
✔️ integration-community.aws-13 SUCCESS in 31m 13s
✔️ integration-community.aws-14 SUCCESS in 13m 22s
✔️ integration-community.aws-15 SUCCESS in 11m 52s
Skipped 7 jobs

@tremble
Copy link
Contributor Author

tremble commented Nov 28, 2023

regate

Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 9m 19s
✔️ build-ansible-collection SUCCESS in 13m 54s
✔️ ansible-test-splitter SUCCESS in 5m 22s
✔️ integration-amazon.aws-1 SUCCESS in 20m 57s
✔️ integration-amazon.aws-2 SUCCESS in 24m 20s
✔️ integration-amazon.aws-3 SUCCESS in 31m 31s
✔️ integration-amazon.aws-4 SUCCESS in 29m 43s
✔️ integration-amazon.aws-5 SUCCESS in 43m 02s
✔️ integration-amazon.aws-6 SUCCESS in 21m 41s
✔️ integration-amazon.aws-7 SUCCESS in 39m 10s
✔️ integration-amazon.aws-8 SUCCESS in 41m 33s
✔️ integration-amazon.aws-9 SUCCESS in 28m 37s
✔️ integration-amazon.aws-10 SUCCESS in 29m 24s
✔️ integration-amazon.aws-11 SUCCESS in 24m 09s
✔️ integration-amazon.aws-12 SUCCESS in 23m 35s
✔️ integration-amazon.aws-13 SUCCESS in 23m 32s
✔️ integration-amazon.aws-14 SUCCESS in 32m 24s
✔️ integration-amazon.aws-15 SUCCESS in 25m 20s
✔️ integration-amazon.aws-16 SUCCESS in 48m 36s
✔️ integration-amazon.aws-17 SUCCESS in 39m 28s
✔️ integration-amazon.aws-18 SUCCESS in 31m 04s
✔️ integration-amazon.aws-19 SUCCESS in 17m 28s
✔️ integration-amazon.aws-20 SUCCESS in 36m 43s
✔️ integration-amazon.aws-21 SUCCESS in 37m 41s
✔️ integration-amazon.aws-22 SUCCESS in 28m 55s
✔️ integration-community.aws-1 SUCCESS in 25m 13s
✔️ integration-community.aws-2 SUCCESS in 49m 34s
✔️ integration-community.aws-3 SUCCESS in 49m 38s
✔️ integration-community.aws-4 SUCCESS in 7m 23s
✔️ integration-community.aws-5 SUCCESS in 22m 20s
✔️ integration-community.aws-6 SUCCESS in 31m 50s
✔️ integration-community.aws-7 SUCCESS in 16m 50s
✔️ integration-community.aws-8 SUCCESS in 7m 34s
✔️ integration-community.aws-9 SUCCESS in 11m 30s
✔️ integration-community.aws-10 SUCCESS in 36m 54s
✔️ integration-community.aws-11 SUCCESS in 19m 12s
✔️ integration-community.aws-12 SUCCESS in 26m 28s
✔️ integration-community.aws-13 SUCCESS in 30m 29s
✔️ integration-community.aws-14 SUCCESS in 10m 57s
✔️ integration-community.aws-15 SUCCESS in 9m 53s
Skipped 7 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 1b55a66 into ansible-collections:main Nov 28, 2023
39 of 40 checks passed
Copy link

patchback bot commented Nov 28, 2023

Backport to stable-7: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-7/1b55a664ffe697aad90042271fcd0ebe5e27d7eb/pr-1876

Backported as #1885

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Nov 28, 2023
Use TagSpecifications parameter when creating EC2 resources

SUMMARY
For the last couple of years Amazon's supported tagging EC2 resources as part of the creation actions.  Switch the last amazon.aws EC2 modules over to using TagSpecifications during creation to support folks using Tagging requirements as part of their IAM/SCP policies
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ec2_vpc_route_table
ec2_vpc_igw
ec2_vpc_subnet
ec2_eip
ADDITIONAL INFORMATION
fixes: #1843

Reviewed-by: Alina Buzachis
(cherry picked from commit 1b55a66)
tremble added a commit that referenced this pull request Nov 29, 2023
…1885)

Use TagSpecifications parameter when creating EC2 resources

SUMMARY
For the last couple of years Amazon's supported tagging EC2 resources as part of the creation actions.  Switch the last amazon.aws EC2 modules over to using TagSpecifications during creation to support folks using Tagging requirements as part of their IAM/SCP policies
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ec2_vpc_route_table
ec2_vpc_igw
ec2_vpc_subnet
ec2_eip
ADDITIONAL INFORMATION
fixes: #1843

Reviewed-by: Alina Buzachis
(cherry picked from commit 1b55a66)

Co-authored-by: Mark Chappell <[email protected]>
@tremble tremble deleted the ec2/resource_tags branch January 12, 2024 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-7 PR should be backported to the stable-7 branch mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tags not included when creating request causes IAM to reject call with IAM tag conditions enabled
2 participants