Skip to content

fix(aws): replace deprecated aws_region.name with .id#2314

Merged
sitole merged 1 commit intoe2b-dev:mainfrom
ya-luotao:fix/aws-deprecated-region-name
Apr 8, 2026
Merged

fix(aws): replace deprecated aws_region.name with .id#2314
sitole merged 1 commit intoe2b-dev:mainfrom
ya-luotao:fix/aws-deprecated-region-name

Conversation

@ya-luotao
Copy link
Copy Markdown
Contributor

Summary

  • Replace deprecated data.aws_region.current.name with data.aws_region.current.id across 4 occurrences in iac/provider-aws/
  • The name attribute was deprecated in AWS Terraform provider v6.x; .id returns the same value (e.g., us-east-1)

Files changed

  • iac/provider-aws/main.tf — 3 occurrences (init module, cluster clickhouse_az, nomad module)
  • iac/provider-aws/alb_athena.tf — 1 occurrence (ALB logs path)

Test plan

  • terraform plan runs without the deprecated attribute warning
  • No changes in planned resources (.name and .id return identical values)

The `name` attribute on `data.aws_region` was deprecated in AWS provider
v6.x. Use the equivalent `id` attribute to silence the warning.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown

@claude claude bot left a comment

Choose a reason for hiding this comment

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

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@sitole sitole merged commit 680d368 into e2b-dev:main Apr 8, 2026
39 of 40 checks passed
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.

3 participants