Skip to content

Commit 0ffb755

Browse files
EMaherCopilot
andcommitted
fix: remove duplicate step properties from merge conflict resolution
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent a70cc67 commit 0ffb755

1 file changed

Lines changed: 7 additions & 22 deletions

File tree

‎.github/workflows/integration-test.yml‎

Lines changed: 7 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -110,21 +110,6 @@ jobs:
110110
echo "::error::Missing required secrets: ${missing[*]}"
111111
exit 1
112112
fi
113-
id: settings
114-
shell: pwsh
115-
run: |
116-
$logLevel = '${{ inputs.log_level }}'
117-
if ([string]::IsNullOrWhiteSpace($logLevel)) { $logLevel = 'Verbose' }
118-
if ($logLevel -notin @('Info', 'Verbose', 'Debug')) {
119-
throw "Invalid log_level '$logLevel'. Allowed values: Info, Verbose, Debug."
120-
}
121-
122-
$skuName = '${{ secrets.APIM_SKU }}'
123-
if ([string]::IsNullOrWhiteSpace($skuName)) { $skuName = '${{ inputs.sku }}' }
124-
if ([string]::IsNullOrWhiteSpace($skuName)) { $skuName = 'StandardV2' }
125-
126-
"logLevel=$logLevel" | Out-File -FilePath $env:GITHUB_OUTPUT -Append
127-
"skuName=$skuName" | Out-File -FilePath $env:GITHUB_OUTPUT -Append
128113
129114
- name: Azure Login
130115
uses: azure/login@v3
@@ -144,9 +129,9 @@ jobs:
144129
TargetApimName = '${{ env.TARGET_APIM }}'
145130
SourceSubscriptionId = '${{ secrets.AZURE_SUBSCRIPTION_ID }}'
146131
TargetSubscriptionId = '${{ secrets.AZURE_SUBSCRIPTION_ID }}'
147-
SkuName = '${{ steps.settings.outputs.skuName }}'
132+
SkuName = '${{ inputs.sku }}'
148133
Location = '${{ inputs.location }}'
149-
LogLevel = '${{ steps.settings.outputs.logLevel }}'
134+
LogLevel = '${{ inputs.log_level }}'
150135
PublisherEmail = '${{ secrets.APIM_PUBLISHER_EMAIL || 'apim-integration-test@contoso.com' }}'
151136
}
152137
./tests/integration/all-resource-types/phases/run-phase1-deploy.ps1 @params
@@ -173,7 +158,7 @@ jobs:
173158
-SourceSubscriptionId '${{ steps.phase1.outputs.sourceSubscriptionId }}' `
174159
-SourceResourceGroup '${{ steps.phase1.outputs.sourceResourceGroup }}' `
175160
-SourceApimName '${{ steps.phase1.outputs.sourceApimName }}' `
176-
-LogLevel '${{ steps.settings.outputs.logLevel }}' `
161+
-LogLevel '${{ inputs.log_level }}' `
177162
-ExtractOutputDir './extracted-artifacts'
178163
179164
- name: Run Round-Trip Phase 3 (Validate Extract)
@@ -185,7 +170,7 @@ jobs:
185170
186171
./tests/integration/all-resource-types/phases/run-phase3-validate-extract.ps1 `
187172
-SkuName $skuName `
188-
-LogLevel '${{ steps.settings.outputs.logLevel }}' `
173+
-LogLevel '${{ inputs.log_level }}' `
189174
-ExtractOutputDir $extractOutputDir
190175
191176
- name: Run Round-Trip Phase 4 (Create Overrides)
@@ -198,7 +183,7 @@ jobs:
198183
./tests/integration/all-resource-types/phases/run-phase4-create-overrides.ps1 `
199184
-TargetSubscriptionId '${{ steps.phase1.outputs.targetSubscriptionId }}' `
200185
-TargetResourceGroup '${{ steps.phase1.outputs.targetResourceGroup }}' `
201-
-LogLevel '${{ steps.settings.outputs.logLevel }}' `
186+
-LogLevel '${{ inputs.log_level }}' `
202187
-ExtractOutputDir $extractOutputDir
203188
204189
- name: Run Round-Trip Phase 5 (Publish)
@@ -215,7 +200,7 @@ jobs:
215200
-TargetResourceGroup '${{ steps.phase1.outputs.targetResourceGroup }}' `
216201
-TargetApimName '${{ steps.phase1.outputs.targetApimName }}' `
217202
-OverrideFile '${{ steps.phase4.outputs.overrideFile }}' `
218-
-LogLevel '${{ steps.settings.outputs.logLevel }}' `
203+
-LogLevel '${{ inputs.log_level }}' `
219204
-ExtractOutputDir $extractOutputDir
220205
221206
- name: Run Round-Trip Phase 6 (Compare)
@@ -229,7 +214,7 @@ jobs:
229214
-TargetSubscriptionId '${{ steps.phase1.outputs.targetSubscriptionId }}' `
230215
-TargetResourceGroup '${{ steps.phase1.outputs.targetResourceGroup }}' `
231216
-TargetApimName '${{ steps.phase1.outputs.targetApimName }}' `
232-
-LogLevel '${{ steps.settings.outputs.logLevel }}'
217+
-LogLevel '${{ inputs.log_level }}'
233218
234219
- name: Run Round-Trip Phase 7 (Teardown)
235220
if: always()

0 commit comments

Comments
 (0)