Skip to content

Commit

Permalink
chore: Apt variable name and false as default
Browse files Browse the repository at this point in the history
  • Loading branch information
k3yss committed Jul 22, 2024
1 parent a9dc4f4 commit 1bdadaa
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion modules/postgresql/gcp/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ resource "google_sql_database_instance" "instance" {
deletion_protection_enabled = !local.destroyable

dynamic "database_flags" {
for_each = local.dms_upgradable ? [{
for_each = local.enable_logical_replication ? [{
name = "cloudsql.logical_decoding"
value = "on"
}, {
Expand Down
6 changes: 3 additions & 3 deletions modules/postgresql/gcp/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -45,10 +45,10 @@ variable "provision_read_replica" {
variable "big_query_connection_location" {
default = "US"
}
variable "dms_upgradable" {
variable "enable_logical_replication" {
description = "Instance is upgradable via Database Migration Service"
type = bool
default = true
default = false
}

locals {
Expand All @@ -66,5 +66,5 @@ locals {
replication = var.replication
provision_read_replica = var.provision_read_replica
big_query_connection_location = var.big_query_connection_location
dms_upgradable = var.dms_upgradable
enable_logical_replication = var.enable_logical_replication
}

0 comments on commit 1bdadaa

Please sign in to comment.