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

[All] Fix tagging when stack and resource level tags have jey collisions #470

Merged

Conversation

khebul
Copy link
Collaborator

@khebul khebul commented Oct 9, 2023

This PR makes sure that Resource level tags always take precedence over stack level tags. If there is a collision between stack and resource level tags, previously erroneous AddTagsToResource or RemoveTagsFromResource requests could be sent. This PR addresses this issue.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@khebul khebul requested review from dbbh and moataz-mhmd October 9, 2023 11:55
@khebul khebul changed the title [All] Make sure that stack level tags do not override resource level tags [All] Fix tagging when stack and resource level tags have jey collisions Oct 9, 2023
dbbh
dbbh previously approved these changes Oct 9, 2023
@khebul khebul merged commit 8c1d2c2 into aws-cloudformation:master Oct 9, 2023
1 check passed
@khebul khebul deleted the resource-stack-level-tags-precedence branch October 9, 2023 16:10
dbbh pushed a commit to dbbh/aws-cloudformation-resource-providers-rds that referenced this pull request Nov 20, 2023
…ons (aws-cloudformation#470)

* [All] Make sure that stack level tags do not override resource level tags

---------

Co-authored-by: Valentin Shirshov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants