Skip to content

feat: added missing launch template tag_specifications #4531

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

Closed
wants to merge 0 commits into from

Conversation

konkerama
Copy link
Contributor

Hello,

Currently the launch template specification adds tags only on resource types of instance and volume this leaves the other 2 supported resource types (spot-instances-request and network-interface) untagged which by extensions does not produce accurate data on Cost Explorer.

This pr fixes this by adding the missing tag_specifications configurations.

@konkerama konkerama requested a review from a team as a code owner April 11, 2025 13:34
Copy link
Member

@npalm npalm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For some reason the main.tf in pool is not formatted propertly. Please can you run a terraform fmt -recursive on the root.

Change looks good to me

@npalm
Copy link
Member

npalm commented Apr 16, 2025

sorry messed up, creating a fresh PR

npalm added a commit that referenced this pull request Apr 16, 2025
copy from #4531

Currently the launch template specification adds tags only on resource
types of `instance` and `volume` this leaves the other 2 supported
resource types (`spot-instances-request` and `network-interface`)
untagged which by extensions does not produce accurate data on Cost
Explorer.

This pr fixes this by adding the missing `tag_specifications`
configurations.

---------

Co-authored-by: konkerama <[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