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

Fix deprecation warnings #160

Merged
merged 2 commits into from
May 23, 2022
Merged

Fix deprecation warnings #160

merged 2 commits into from
May 23, 2022

Conversation

purplepangolin
Copy link
Contributor

@purplepangolin purplepangolin commented Apr 29, 2022

what

  • Replaces use of deprecated attributes (cluster_mode, replication_group_description, number_cache_clusters) in aws_elasticache_replication_group resurce when using provider registry.terraform.io/hashicorp/aws v4.12.0

why

Eliminate warnings when running terraform plan by moving to latest supported attributes instead.

references

Terraform aws provider docs

@purplepangolin purplepangolin requested review from a team as code owners April 29, 2022 10:41
@purplepangolin purplepangolin requested review from dylanbannon and brcnblc and removed request for a team April 29, 2022 10:41
@aknysh
Copy link
Member

aknysh commented May 23, 2022

/test all

Copy link
Member

@aknysh aknysh left a comment

Choose a reason for hiding this comment

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

@aknysh aknysh merged commit 23557c5 into cloudposse:master May 23, 2022
@syphernl
Copy link
Contributor

Shouldn't this have been marked as a breaking change and/or also updated the minimum AWS provider version?

This is causing issues with v3 of the AWS provider (hashicorp/aws v3.75.2 to be exact):

│ Error: Missing required argument
│ 
│   on .terraform/modules/redis/main.tf line 116, in resource "aws_elasticache_replication_group" "default":
│  116: resource "aws_elasticache_replication_group" "default" {
│ 
│ The argument "replication_group_description" is required, but no definition was found.
╵
╷
│ Error: Unsupported argument
│ 
│   on .terraform/modules/redis/main.tf line 121, in resource "aws_elasticache_replication_group" "default":
│  121:   description                = module.this.id
│ 
│ An argument named "description" is not expected here.
╵
╷
│ Error: Unsupported argument
│ 
│   on .terraform/modules/redis/main.tf line 123, in resource "aws_elasticache_replication_group" "default":
│  123:   num_cache_clusters         = var.cluster_mode_enabled ? null : var.cluster_size
│ 
│ An argument named "num_cache_clusters" is not expected here.
╵
╷
│ Error: Unsupported argument
│ 
│   on .terraform/modules/redis/main.tf line 149, in resource "aws_elasticache_replication_group" "default":
│  149:   num_node_groups         = var.cluster_mode_enabled ? var.cluster_mode_num_node_groups : null
│ 
│ An argument named "num_node_groups" is not expected here.
╵
╷
│ Error: Unsupported argument
│ 
│   on .terraform/modules/redis/main.tf line 150, in resource "aws_elasticache_replication_group" "default":
│  150:   replicas_per_node_group = var.cluster_mode_enabled ? var.cluster_mode_replicas_per_node_group : null
│ 
│ An argument named "replicas_per_node_group" is not expected here.

ismailyenigul added a commit to ismailyenigul/terraform-aws-elasticache-redis that referenced this pull request May 26, 2022
Update minimum version of terraform aws provider to support num_cache_clusters
cloudposse#160 (comment)
brian-weis-msr pushed a commit to Measurabl/terraform-aws-elasticache-redis that referenced this pull request Apr 2, 2024
* Fix deprecation warnings

* Auto Format

Co-authored-by: cloudpossebot <11232728+cloudpossebot@users.noreply.github.com>
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.

Argument is deprecated [elasticache_replication_group]
4 participants