From 514babbba8f656b1ae903473726b10dd3163b109 Mon Sep 17 00:00:00 2001 From: Ben Broderick Phillips Date: Tue, 4 Jun 2024 19:07:20 -0400 Subject: [PATCH 1/3] Plumb env vars through deploy test resources --- eng/common/TestResources/deploy-test-resources.yml | 5 +++++ eng/common/TestResources/remove-test-resources.yml | 3 +++ 2 files changed, 8 insertions(+) diff --git a/eng/common/TestResources/deploy-test-resources.yml b/eng/common/TestResources/deploy-test-resources.yml index b4197dc236d9..0f5b1c339663 100644 --- a/eng/common/TestResources/deploy-test-resources.yml +++ b/eng/common/TestResources/deploy-test-resources.yml @@ -3,6 +3,7 @@ parameters: ArmTemplateParameters: '@{}' DeleteAfterHours: 8 Location: '' + EnvVars: {} SubscriptionConfiguration: $(sub-config-azure-cloud-test-resources) ServiceConnection: not-specified ResourceType: test @@ -47,6 +48,8 @@ steps: displayName: Deploy test resources env: TEMP: $(Agent.TempDirectory) + ${{ each var in parameters.EnvVars }}: + ${{ var.key }}: ${{ var.value }} inputs: azureSubscription: ${{ parameters.ServiceConnection }} azurePowerShellVersion: LatestVersion @@ -106,3 +109,5 @@ steps: displayName: Deploy test resources env: TEMP: $(Agent.TempDirectory) + ${{ each var in parameters.EnvVars }}: + ${{ var.key }}: ${{ var.value }} diff --git a/eng/common/TestResources/remove-test-resources.yml b/eng/common/TestResources/remove-test-resources.yml index 89aa7d825e48..5009eb53f2a9 100644 --- a/eng/common/TestResources/remove-test-resources.yml +++ b/eng/common/TestResources/remove-test-resources.yml @@ -6,6 +6,7 @@ parameters: SubscriptionConfiguration: $(sub-config-azure-cloud-test-resources) ServiceConnection: not-specified ResourceType: test + EnvVars: {} UseFederatedAuth: false SubscriptionConfigurationFilePath: '' @@ -29,6 +30,7 @@ steps: displayName: Remove test resources condition: and(eq(variables['CI_HAS_DEPLOYED_RESOURCES'], 'true'), ne(variables['Skip.RemoveTestResources'], 'true')) continueOnError: true + env: ${{ parameters.EnvVars }} inputs: azureSubscription: ${{ parameters.ServiceConnection }} azurePowerShellVersion: LatestVersion @@ -78,3 +80,4 @@ steps: displayName: Remove test resources condition: and(eq(variables['CI_HAS_DEPLOYED_RESOURCES'], 'true'), ne(variables['Skip.RemoveTestResources'], 'true')) continueOnError: true + env: ${{ parameters.EnvVars }} From 3037ae40a2b85dd73c8d87e043f0bcdd7c3cdfb8 Mon Sep 17 00:00:00 2001 From: Ben Broderick Phillips Date: Wed, 5 Jun 2024 17:36:23 -0400 Subject: [PATCH 2/3] Update eng/common/TestResources/deploy-test-resources.yml Co-authored-by: Daniel Jurek --- eng/common/TestResources/deploy-test-resources.yml | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/eng/common/TestResources/deploy-test-resources.yml b/eng/common/TestResources/deploy-test-resources.yml index 0f5b1c339663..d9efad975755 100644 --- a/eng/common/TestResources/deploy-test-resources.yml +++ b/eng/common/TestResources/deploy-test-resources.yml @@ -48,8 +48,7 @@ steps: displayName: Deploy test resources env: TEMP: $(Agent.TempDirectory) - ${{ each var in parameters.EnvVars }}: - ${{ var.key }}: ${{ var.value }} + ${{ insert }}: ${{ parameters.EnvVars }} inputs: azureSubscription: ${{ parameters.ServiceConnection }} azurePowerShellVersion: LatestVersion From 862ebc1d172f5df22578fd0b9283d8db83ccf66a Mon Sep 17 00:00:00 2001 From: Ben Broderick Phillips Date: Wed, 5 Jun 2024 17:36:29 -0400 Subject: [PATCH 3/3] Update eng/common/TestResources/deploy-test-resources.yml Co-authored-by: Daniel Jurek --- eng/common/TestResources/deploy-test-resources.yml | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/eng/common/TestResources/deploy-test-resources.yml b/eng/common/TestResources/deploy-test-resources.yml index d9efad975755..724dfd8addb9 100644 --- a/eng/common/TestResources/deploy-test-resources.yml +++ b/eng/common/TestResources/deploy-test-resources.yml @@ -108,5 +108,4 @@ steps: displayName: Deploy test resources env: TEMP: $(Agent.TempDirectory) - ${{ each var in parameters.EnvVars }}: - ${{ var.key }}: ${{ var.value }} + ${{ insert }}: ${{ parameters.EnvVars }}