Skip to content

fix(google_cloudsql_mysql): switching from coalesce to concat#435

Merged
izo3 merged 1 commit intomainfrom
SVCSE-4188
Feb 5, 2026
Merged

fix(google_cloudsql_mysql): switching from coalesce to concat#435
izo3 merged 1 commit intomainfrom
SVCSE-4188

Conversation

@izo3
Copy link
Copy Markdown
Contributor

@izo3 izo3 commented Feb 3, 2026

Description

Replace coalescelist with concat in replica_names local to prevent "no non-null arguments" error when replica_count = 0 and custom_replica_name is empty (non-prod default). concat safely returns empty list instead of failing.

For example under common-voice:

│ 
│   on .terraform/modules/database.mysql_database/google_cloudsql_mysql/main.tf line 15, in locals:
│   15:   replica_names        = coalescelist(var.custom_replica_name, [for index in range(var.replica_count) : "${local.default_replica_name}-${index}"])
│     ├────────────────
│     │ while calling coalescelist(vals...)
│     │ local.default_replica_name is "common-voice-nonprod-sandbox-v2-replica"
│     │ var.custom_replica_name is empty list of string
│     │ var.replica_count is 0
│ 
│ Call to function "coalescelist" failed: no non-null arguments.

Related Tickets & Documents

@github-actions github-actions Bot added the patch This PR will increment a patch version label Feb 3, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 3, 2026

Release plan

Directory Previous version New version
google_cloudsql_mysql 1.0.0 1.0.1

@izo3 izo3 merged commit 161f87e into main Feb 5, 2026
10 checks passed
@izo3 izo3 deleted the SVCSE-4188 branch February 5, 2026 16:25
whd pushed a commit that referenced this pull request Feb 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

patch This PR will increment a patch version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants