Skip to content

Commit

Permalink
fix: upgrade aws provider to 5.29.0 to support storage_type (#103)
Browse files Browse the repository at this point in the history
* build: upgrade aws provider to 5.29.0

* Update examples/complete/variables.tf

Co-authored-by: Veronika Gnilitska <[email protected]>

---------

Co-authored-by: Veronika Gnilitska <[email protected]>
  • Loading branch information
gmeligio and gberenice authored Jul 25, 2024
1 parent 19f58f4 commit 6fc2bac
Show file tree
Hide file tree
Showing 6 changed files with 19 additions and 5 deletions.
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -102,15 +102,15 @@ Available targets:
| Name | Version |
|------|---------|
| <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 1.3 |
| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 5.21.0 |
| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 5.29.0 |
| <a name="requirement_local"></a> [local](#requirement\_local) | >= 1.3 |
| <a name="requirement_random"></a> [random](#requirement\_random) | >= 1.0 |

## Providers

| Name | Version |
|------|---------|
| <a name="provider_aws"></a> [aws](#provider\_aws) | >= 5.21.0 |
| <a name="provider_aws"></a> [aws](#provider\_aws) | >= 5.29.0 |
| <a name="provider_random"></a> [random](#provider\_random) | >= 1.0 |

## Modules
Expand Down
4 changes: 2 additions & 2 deletions docs/terraform.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,15 +4,15 @@
| Name | Version |
|------|---------|
| <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 1.3 |
| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 5.21.0 |
| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 5.29.0 |
| <a name="requirement_local"></a> [local](#requirement\_local) | >= 1.3 |
| <a name="requirement_random"></a> [random](#requirement\_random) | >= 1.0 |

## Providers

| Name | Version |
|------|---------|
| <a name="provider_aws"></a> [aws](#provider\_aws) | >= 5.21.0 |
| <a name="provider_aws"></a> [aws](#provider\_aws) | >= 5.29.0 |
| <a name="provider_random"></a> [random](#provider\_random) | >= 1.0 |

## Modules
Expand Down
2 changes: 2 additions & 0 deletions examples/complete/fixtures.us-east-2.tfvars
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,8 @@ engine = "docdb"

storage_encrypted = true

storage_type = "standard"

skip_final_snapshot = true

apply_immediately = true
Expand Down
1 change: 1 addition & 0 deletions examples/complete/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,7 @@ module "documentdb_cluster" {
engine = var.engine
engine_version = var.engine_version
storage_encrypted = var.storage_encrypted
storage_type = var.storage_type
kms_key_id = var.kms_key_id
skip_final_snapshot = var.skip_final_snapshot
enabled_cloudwatch_logs_exports = var.enabled_cloudwatch_logs_exports
Expand Down
11 changes: 11 additions & 0 deletions examples/complete/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -119,6 +119,17 @@ variable "storage_encrypted" {
default = true
}

variable "storage_type" {
type = string
description = "The storage type to associate with the DB cluster. Valid values: standard, iopt1"
default = "standard"

validation {
condition = contains(["standard", "iopt1"], var.storage_type)
error_message = "Error: storage_type value must be one of two options - 'standard' or 'iopt1'."
}
}

variable "kms_key_id" {
type = string
description = "The ARN for the KMS encryption key. When specifying `kms_key_id`, `storage_encrypted` needs to be set to `true`"
Expand Down
2 changes: 1 addition & 1 deletion versions.tf
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ terraform {
required_providers {
aws = {
source = "hashicorp/aws"
version = ">= 5.21.0"
version = ">= 5.29.0"
}
local = {
source = "hashicorp/local"
Expand Down

0 comments on commit 6fc2bac

Please sign in to comment.