Skip to content

Commit 15ec774

Browse files
committed
ci: run full Azure integration suite under OIDC
With provider pins modernized in the prior commits, the Azure integration workflow can run the full test/azure/... suite under OIDC auth. Removes the brittle enumerated -run regex flagged in #1794 review feedback: #1794 (comment) Also: - Flips concurrency.cancel-in-progress to false so a cancelled run can't orphan Azure resources mid `terraform apply`. - Bumps go test -timeout from 45m to 120m and job timeout-minutes from 60 to 150; the full sequential (-p 1) suite of ~28 tests can take 60-90 minutes (heavier services like AKS/Synapse/SQL MI/MySQL flexible-server provision in 10+ minutes each).
1 parent f72d4d2 commit 15ec774

1 file changed

Lines changed: 7 additions & 7 deletions

File tree

.github/workflows/azure-integration-tests.yml

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,9 @@ on:
2424

2525
concurrency:
2626
group: azure-integration-tests-${{ github.ref }}
27-
cancel-in-progress: true
27+
# Don't cancel in-flight runs; tests provision real resources and rely
28+
# on `defer terraform.Destroy` for cleanup.
29+
cancel-in-progress: false
2830

2931
jobs:
3032
azure-integration-tests:
@@ -33,7 +35,7 @@ jobs:
3335
github.event_name != 'pull_request' ||
3436
github.event.pull_request.head.repo.full_name == github.repository
3537
runs-on: ubuntu-latest
36-
timeout-minutes: 60
38+
timeout-minutes: 150
3739
permissions:
3840
id-token: write
3941
contents: read
@@ -78,11 +80,9 @@ jobs:
7880
run: |
7981
set -o pipefail
8082
mkdir -p /tmp/logs
81-
# Scope: OIDC-compatible examples only (AzureRM provider >= 3.7).
82-
# Examples on AzureRM 2.x cannot authenticate via OIDC and are
83-
# gated behind a follow-up provider-upgrade effort.
84-
go test -v -p 1 -tags azure -count=1 -timeout 45m \
85-
-run '^(TestTerraformAzureContainerAppExample|TestTerraformAzureKeyVaultExample)$' \
83+
# Run the full Azure integration suite. All examples are pinned to
84+
# AzureRM >= 3.x and authenticate via OIDC.
85+
go test -v -p 1 -tags azure -count=1 -timeout 120m \
8686
./test/azure/... 2>&1 | tee /tmp/logs/azure-integration-tests.log
8787
8888
- name: Upload test logs

0 commit comments

Comments
 (0)