Skip to content

Commit 38b4b85

Browse files
authored
Removing skip teardown option from integration test workflow (#91)
* Remove skip_teardown option from integration-test workflow * Update integration-test defaults and branch gating * Restore run-roundtrip-test script and keep SkipTeardown removal in workflow only * Revert default region from eastus back to centralus --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
1 parent cbe134f commit 38b4b85

1 file changed

Lines changed: 2 additions & 14 deletions

File tree

.github/workflows/integration-test.yml

Lines changed: 2 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,6 @@ on:
3232
type: choice
3333
options:
3434
- StandardV2
35-
- Developer
3635
- Premium
3736
- PremiumV2
3837
default: StandardV2
@@ -46,11 +45,6 @@ on:
4645
required: false
4746
type: string
4847
default: Verbose
49-
skip_teardown:
50-
description: 'Skip teardown (for debugging)'
51-
required: false
52-
type: boolean
53-
default: false
5448

5549
workflow_call:
5650
inputs:
@@ -69,11 +63,6 @@ on:
6963
required: false
7064
type: string
7165
default: Verbose
72-
skip_teardown:
73-
description: 'Skip teardown (for debugging)'
74-
required: false
75-
type: boolean
76-
default: false
7766
secrets:
7867
AZURE_CLIENT_ID:
7968
required: true
@@ -99,6 +88,7 @@ env:
9988
jobs:
10089
roundtrip-test:
10190
name: Extract→Publish Round-Trip
91+
if: github.ref == 'refs/heads/main'
10292
runs-on: ubuntu-latest
10393
timeout-minutes: 120
10494
environment: integration-test # Requires approval for cost protection
@@ -123,7 +113,6 @@ jobs:
123113
- name: Run Round-Trip Test
124114
shell: pwsh
125115
run: |
126-
$skipTeardown = '${{ inputs.skip_teardown }}' -eq 'true'
127116
$logLevel = '${{ inputs.log_level }}'
128117
if ([string]::IsNullOrWhiteSpace($logLevel)) { $logLevel = 'Verbose' }
129118
if ($logLevel -notin @('Info', 'Verbose', 'Debug')) {
@@ -140,7 +129,6 @@ jobs:
140129
ExtractOutputDir = './extracted-artifacts'
141130
HardDelete = $true
142131
}
143-
if ($skipTeardown) { $params.SkipTeardown = $true }
144132
145133
./tests/integration/all-resource-types/run-roundtrip-test.ps1 @params
146134
@@ -153,7 +141,7 @@ jobs:
153141
if-no-files-found: ignore
154142

155143
- name: Emergency Teardown
156-
if: failure() && inputs.skip_teardown != true
144+
if: failure()
157145
shell: pwsh
158146
run: |
159147
$sourceRg = '${{ env.SOURCE_RG }}'

0 commit comments

Comments
 (0)