From ccad5654d9aa2f5df9b20ad9b3f2b98fb4d6ae50 Mon Sep 17 00:00:00 2001 From: ooshrioo <98342127+ooshrioo@users.noreply.github.com> Date: Wed, 7 Jan 2026 15:02:13 +0200 Subject: [PATCH] Fix workforce identity federation provider configuration (#3626) * Fix workforce identity federation provider configuration Remove redundant 'organizations/' prefix from parent parameter as var.organization_id already contains the full organization path. * Update test to match corrected parent parameter The test expected the redundant 'organizations/' prefix, but the fix correctly removes this duplication. --------- Co-authored-by: Julio Castillo --- modules/organization/identity-providers.tf | 2 +- tests/modules/organization/examples/wfif.yaml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/modules/organization/identity-providers.tf b/modules/organization/identity-providers.tf index 533c4c9c9..91dd5818e 100644 --- a/modules/organization/identity-providers.tf +++ b/modules/organization/identity-providers.tf @@ -39,7 +39,7 @@ locals { resource "google_iam_workforce_pool" "default" { count = var.workforce_identity_config == null ? 0 : 1 - parent = "organizations/${var.organization_id}" + parent = var.organization_id location = "global" workforce_pool_id = var.workforce_identity_config.pool_name } diff --git a/tests/modules/organization/examples/wfif.yaml b/tests/modules/organization/examples/wfif.yaml index 411e3b6b3..d29c0ba88 100644 --- a/tests/modules/organization/examples/wfif.yaml +++ b/tests/modules/organization/examples/wfif.yaml @@ -19,7 +19,7 @@ values: disabled: null display_name: null location: global - parent: organizations/organizations/1122334455 + parent: organizations/1122334455 session_duration: 3600s timeouts: null workforce_pool_id: test-pool