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

feat: add tags support to root block device #218

Merged

Conversation

achachw
Copy link
Contributor

@achachw achachw commented May 11, 2021

Description

Adding tag support to root block device

Motivation and Context

Actually, all of these parameters are supported for root_block_device block : volume_size, volume_type, iops, .. except tags.
I would suggest to add tags.

Breaking Changes

How Has This Been Tested?

  • I have tested and validated these changes using one or more of the provided examples/* projects

@antonbabenko
Copy link
Member

Hi,

tags became available in 3.24.0 - https://github.com/hashicorp/terraform-provider-aws/blob/main/CHANGELOG.md#3240-january-14-2021

Please upgrade versions.tf accordingly.

@achachw
Copy link
Contributor Author

achachw commented May 11, 2021

Hi @antonbabenko
Thanks for your quick reply.

main.tf Outdated Show resolved Hide resolved
@antonbabenko antonbabenko merged commit c49475f into terraform-aws-modules:master May 11, 2021
@antonbabenko
Copy link
Member

v2.18.0 has been just released.

@jinglejengel
Copy link

FYI this appears to have broken the module.

When trying both no tags in the root_block_device and passing in a map of strings to a tag field in the root_block_device I receive the same error:

Error: Incorrect attribute value type

  on .terraform/modules/ec2/main.tf line 38, in resource "aws_instance" "this":
  38:       tags                  = lookup(root_block_device.value, "tags", null)

Inappropriate value for attribute "tags": map of string required.

@antonbabenko
Copy link
Member

Too bad that I trusted this PR without actually doing the normal procedure. Fixed in #220.

Please update to use v2.19.0 that has been just released.

@jinglejengel
Copy link

Thanks for super quick turnaround @antonbabenko!

@yatryan
Copy link

yatryan commented May 14, 2021

Unfortunately, this update was a breaking change for a number of my projects. Bumping up hashicorp/aws from ">= 2.65" to ">= 3.24" caused some incompatibilities. Any reason this wasn't a breaking change version bump?

@achachw
Copy link
Contributor Author

achachw commented May 17, 2021

Ah sorry for this miss .. guys.

@grig-tar
Copy link

grig-tar commented Aug 2, 2021

One more issue here - tags were added to 'root_block_device' only but forgotten about 'ebs_block_device'.

@github-actions
Copy link

github-actions bot commented Nov 8, 2022

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants