Skip to content

Commit aa89133

Browse files
committed
Merge EnvVars correctly, only include SubscriptionConfigurationFilePaths if using federated auth
1 parent 5534fde commit aa89133

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

eng/pipelines/templates/jobs/live.tests.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -162,8 +162,9 @@ jobs:
162162
SubscriptionConfigurations: ${{ parameters.CloudConfig.SubscriptionConfigurations }}
163163
EnvVars:
164164
Pool: $(Pool)
165-
SubscriptionConfigurationFilePaths: ${{ parameters.CloudConfig.SubscriptionConfigurationFilePaths }}
166-
EnvVars: ${{ parameters.EnvVars }}
165+
${{ insert }}: ${{ parameters.EnvVars }}
166+
${{ if parameters.UseFederatedAuth }}:
167+
SubscriptionConfigurationFilePaths: ${{ parameters.CloudConfig.SubscriptionConfigurationFilePaths }}
167168

168169
- template: /eng/common/TestResources/deploy-test-resources.yml
169170
parameters:
@@ -172,8 +173,8 @@ jobs:
172173
SubscriptionConfiguration: $(SubscriptionConfiguration)
173174
EnvVars:
174175
Pool: $(Pool)
176+
${{ insert }}: ${{ parameters.EnvVars }}
175177
UseFederatedAuth: ${{ parameters.UseFederatedAuth }}
176-
EnvVars: ${{ parameters.EnvVars }}
177178
ServiceConnection: ${{ parameters.CloudConfig.ServiceConnection }}
178179

179180
- template: /eng/common/testproxy/test-proxy-tool.yml

0 commit comments

Comments
 (0)