Skip to content

Commit

Permalink
Merge pull request #65 from schubergphilis/bug-coalesce
Browse files Browse the repository at this point in the history
fix: solve issue with using coalesce on null values
  • Loading branch information
marwinbaumannsbp authored Jan 10, 2025
2 parents c5cbb18 + 53a6d77 commit 5ec1e77
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -244,11 +244,11 @@ module "additional_tfe_workspaces" {
description = each.value.description
execution_mode = coalesce(each.value.execution_mode, var.tfe_workspace.execution_mode)
file_triggers_enabled = each.value.connect_vcs_repo != false ? each.value.file_triggers_enabled : null
github_app_installation_id = each.value.connect_vcs_repo != false ? coalesce(each.value.vcs_github_app_installation_id, var.tfe_workspace.vcs_github_app_installation_id) : null
github_app_installation_id = each.value.connect_vcs_repo != false ? try(coalesce(each.value.vcs_github_app_installation_id, var.tfe_workspace.vcs_github_app_installation_id), null) : null
global_remote_state = each.value.global_remote_state
name = coalesce(each.value.name, each.key)
notification_configuration = each.value.notification_configuration != null ? each.value.notification_configuration : var.tfe_workspace.notification_configuration
oauth_token_id = each.value.connect_vcs_repo != false ? coalesce(each.value.vcs_oauth_token_id, var.tfe_workspace.vcs_oauth_token_id) : null
oauth_token_id = each.value.connect_vcs_repo != false ? try(coalesce(each.value.vcs_oauth_token_id, var.tfe_workspace.vcs_oauth_token_id), null) : null
oidc_settings = coalesce(each.value.auth_method, var.tfe_workspace.auth_method) == "iam_role_oidc" ? { provider_arn = aws_iam_openid_connect_provider.tfc_provider[0].arn } : null
path = var.path
permissions_boundary_arn = each.value.add_permissions_boundary == true ? aws_iam_policy.workspace_boundary[0].arn : null
Expand Down

0 comments on commit 5ec1e77

Please sign in to comment.