Skip to content

Commit

Permalink
Merge branch 'main' into msk-logging-change, resolve mc
Browse files Browse the repository at this point in the history
  • Loading branch information
jerryfan01234 committed Aug 12, 2024
2 parents b3d6a0a + 180d70c commit d86d0ab
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion indexer/msk.tf
Original file line number Diff line number Diff line change
@@ -1,8 +1,14 @@
resource "aws_msk_configuration" "main" {
<<<<<<< HEAD
// create_before_destroy=true requires us to use name whenever this resource needs replacement because the old
// resource not delete first. For now we only trigger replacement for new kafka versions, so use the kafka version in
// the name
kafka_versions = [local.kafka_version]
=======
kafka_versions = [local.kafka_version]
// create_before_destroy=true forces a new name because the old resource not delete first. For now we only trigger
// replacement for new kafka versions, so use the kafka version in the name
>>>>>>> main
name = "${var.environment}-${var.indexers[var.region].name}-msk-configuration-${replace(local.kafka_version, ".", "-")}"
server_properties = <<PROPERTIES
auto.create.topics.enable=false
Expand Down Expand Up @@ -57,4 +63,4 @@ resource "aws_msk_cluster" "main" {
arn = aws_msk_configuration.main.arn
revision = aws_msk_configuration.main.latest_revision
}
}
}

0 comments on commit d86d0ab

Please sign in to comment.