fix: Allow setting master_username when using `manage_master_user_p…

…assword` (#369)
terraform-aws-modules · Apr 11, 2023 · f48f824 · f48f824
1 parent 0dd4a5c
commit f48f824
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -1,6 +1,6 @@
 repos:
   - repo: https://github.com/antonbabenko/pre-commit-terraform
-    rev: v1.77.1
+    rev: v1.77.2
     hooks:
       - id: terraform_fmt
       - id: terraform_validate

diff --git a/main.tf b/main.tf
@@ -68,7 +68,7 @@ resource "aws_rds_cluster" "this" {
   manage_master_user_password   = var.global_cluster_identifier == null && var.manage_master_user_password ? var.manage_master_user_password : null
   master_user_secret_kms_key_id = var.global_cluster_identifier == null && var.manage_master_user_password ? var.master_user_secret_kms_key_id : null
   master_password               = var.is_primary_cluster && !var.manage_master_user_password ? var.master_password : null
-  master_username               = var.is_primary_cluster && !var.manage_master_user_password ? var.master_username : null
+  master_username               = var.is_primary_cluster ? var.master_username : null
   network_type                  = var.network_type
   port                          = local.port
   preferred_backup_window       = local.is_serverless ? null : var.preferred_backup_window