Skip to content

Commit b808128

Browse files
committed
CCM-10231: Review comments
1 parent 6aedaf0 commit b808128

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

infrastructure/modules/kms/kms_replica_key_replica.tf

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
resource "aws_kms_replica_key" "replica" {
2-
count = var.is_multi_region ? 1 : 0
32
provider = aws.us-east-1
3+
count = var.is_multi_region ? 1 : 0
4+
45
description = "Multi-Region replica key"
56
deletion_window_in_days = var.deletion_window
67
policy = data.aws_iam_policy_document.key.json

infrastructure/modules/kms/variables.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,6 @@ variable "iam_delegation" {
6969

7070
variable "is_multi_region" {
7171
type = bool
72-
description = "Whether the KMS key is a multi-region key. Defaults to false"
72+
description = "Whether the KMS key is a multi-region key, where secondary region would mostly be us-east-1. Defaults to false"
7373
default = false
7474
}

0 commit comments

Comments
 (0)