Skip to content

Commit 8c6b85c

Browse files
authored
fix: address release workflow review feedback
1 parent 940bc9c commit 8c6b85c

1 file changed

Lines changed: 20 additions & 11 deletions

File tree

.github/workflows/release-tests.yml

Lines changed: 20 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -35,38 +35,47 @@ on:
3535
default: Verbose
3636

3737
permissions:
38-
id-token: write
3938
contents: read
4039

4140
jobs:
41+
validate-main:
42+
name: Validate Main Branch
43+
runs-on: ubuntu-latest
44+
steps:
45+
- name: Require refs/heads/main
46+
run: |
47+
if [ "${GITHUB_REF}" != "refs/heads/main" ]; then
48+
echo "Release Tests must run from refs/heads/main; received ${GITHUB_REF}." >&2
49+
exit 1
50+
fi
51+
4252
ci:
4353
name: CI (Unit + Build Package Tests)
54+
needs: validate-main
4455
uses: ./.github/workflows/ci.yml
4556

4657
all-types-integration:
4758
name: All-Types Integration Test
4859
needs: ci
60+
permissions:
61+
id-token: write
62+
contents: read
4963
uses: ./.github/workflows/integration-test.yml
5064
with:
5165
sku: ${{ inputs.sku }}
5266
location: ${{ inputs.location }}
5367
log_level: ${{ inputs.log_level }}
54-
secrets:
55-
AZURE_CLIENT_ID: ${{ secrets.AZURE_CLIENT_ID }}
56-
AZURE_TENANT_ID: ${{ secrets.AZURE_TENANT_ID }}
57-
AZURE_SUBSCRIPTION_ID: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
58-
APIM_PUBLISHER_EMAIL: ${{ secrets.APIM_PUBLISHER_EMAIL }}
68+
secrets: inherit
5969

6070
redact-secrets-integration:
6171
name: Redact Secrets Integration Test
6272
needs: all-types-integration
73+
permissions:
74+
id-token: write
75+
contents: read
6376
uses: ./.github/workflows/integration-redact-secrets.yml
6477
with:
6578
sku: ${{ inputs.sku }}
6679
location: ${{ inputs.location }}
6780
log_level: ${{ inputs.log_level }}
68-
secrets:
69-
AZURE_CLIENT_ID: ${{ secrets.AZURE_CLIENT_ID }}
70-
AZURE_TENANT_ID: ${{ secrets.AZURE_TENANT_ID }}
71-
AZURE_SUBSCRIPTION_ID: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
72-
APIM_PUBLISHER_EMAIL: ${{ secrets.APIM_PUBLISHER_EMAIL }}
81+
secrets: inherit

0 commit comments

Comments
 (0)