Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

multivpc. fix "NPE" in replace #2586

Merged
merged 1 commit into from
May 14, 2024

Conversation

mr0re1
Copy link
Collaborator

@mr0re1 mr0re1 commented May 14, 2024

Fix

╷
│ Error: Invalid function argument
│
│   on modules/embedded/modules/network/multivpc/main.tf line 19, in locals:
│   19:   network_name          = coalesce(replace(var.network_name_prefix, "_", "-"), replace(var.deployment_name, "_", "-"))
│     ├────────────────
│     │ while calling replace(str, substr, replace)
│     │ var.network_name_prefix is null
│
│ Invalid value for "str" parameter: argument must not be null.

@mr0re1 mr0re1 added the release-chore To not include into release notes label May 14, 2024
@mr0re1 mr0re1 requested a review from tpdownes May 14, 2024 00:08
@mr0re1 mr0re1 changed the title Fix "NPE" in replace multivpc. fix "NPE" in replace May 14, 2024
@tpdownes tpdownes assigned mr0re1 and unassigned tpdownes May 14, 2024
@mr0re1 mr0re1 merged commit af2e641 into GoogleCloudPlatform:develop May 14, 2024
12 of 52 checks passed
@mr0re1 mr0re1 deleted the multivpc_npe branch May 14, 2024 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-chore To not include into release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants