Skip to content

feat: Upgrade AWS provider to v6 and remove deprecated arguments: elastic_gpu_specifications and elastic_inference_accelerator #3406

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

Conversation

dennisameling
Copy link

@dennisameling dennisameling commented Jul 7, 2025

Description

We forked this repo to check what's needed for it to work with the AWS v6 provider. In our testing, we only ran into problems with the elastic_gpu_specifications and elastic_inference_accelerator arguments:

╷
│ Error: Unsupported block type
│ 
│   on .terraform/modules/eks.k8s_cluster/modules/eks-managed-node-group/main.tf line 140, in resource "aws_launch_template" "this":
│  140:   dynamic "elastic_gpu_specifications" {
│ 
│ Blocks of type "elastic_gpu_specifications" are not expected here.
╵
╷
│ Error: Unsupported block type
│ 
│   on .terraform/modules/eks.k8s_cluster/modules/eks-managed-node-group/main.tf line 148, in resource "aws_launch_template" "this":
│  148:   dynamic "elastic_inference_accelerator" {
│ 
│ Blocks of type "elastic_inference_accelerator" are not expected here.
╵
╷
│ Error: Unsupported block type
│ 
│   on .terraform/modules/eks.k8s_cluster/modules/self-managed-node-group/main.tf line 203, in resource "aws_launch_template" "this":
│  203:   dynamic "elastic_gpu_specifications" {
│ 
│ Blocks of type "elastic_gpu_specifications" are not expected here.
╵
╷
│ Error: Unsupported block type
│ 
│   on .terraform/modules/eks.k8s_cluster/modules/self-managed-node-group/main.tf line 211, in resource "aws_launch_template" "this":
│  211:   dynamic "elastic_inference_accelerator" {
│ 
│ Blocks of type "elastic_inference_accelerator" are not expected here.

Once we removed these, our setup worked right away with v6 of the AWS provider.

Motivation and Context

Hopefully this helps in the path to AWS provider v6 support: #3386

Breaking Changes

This is a breaking change, because users of this module can no longer provide elastic_gpu_specifications and elastic_inference_accelerator variables to the module.

This addresses some breaking changes in v6 of the AWS provider:

How Has This Been Tested?

We are running these changes in our production environment with the AWS provider v6 and can confirm that everything's working as expected.

I'll file a separate PR to bump the hashicorp/aws version to >= 6.0.0 in the main module and the examples.

  • I have updated at least one of the examples/* to demonstrate and validate my change(s)
  • I have tested and validated these changes using one or more of the provided examples/* projects
  • I have executed pre-commit run -a on my pull request

@dennisameling dennisameling changed the title Remove deprecated AWS provider v6 arguments: elastic_gpu_specifications and elastic_inference_accelerator feat: upgrade AWS provider to v6 and remove deprecated arguments: elastic_gpu_specifications and elastic_inference_accelerator Jul 7, 2025
@dennisameling dennisameling changed the title feat: upgrade AWS provider to v6 and remove deprecated arguments: elastic_gpu_specifications and elastic_inference_accelerator feat: Upgrade AWS provider to v6 and remove deprecated arguments: elastic_gpu_specifications and elastic_inference_accelerator Jul 7, 2025
@dennisameling
Copy link
Author

Just went through all of the folders in examples/* and did a terraform plan on each of them in our environment. All looking good, no errors popped up! 🚀

@dennisameling
Copy link
Author

Closing in favor of #3385. At least good to see that things are working as expected on v6 of the provider.

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.

1 participant