Skip to content

Commit cf0b4ec

Browse files
committed
feat: update Azure login step to clarify tenant-only usage for Graph API
1 parent 08ba84d commit cf0b4ec

1 file changed

Lines changed: 5 additions & 2 deletions

File tree

.github/workflows/teardown.yml

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -185,13 +185,16 @@ jobs:
185185
echo "spa_app_id=$spa_app_id" >> "$GITHUB_OUTPUT"
186186
echo "skip=false" >> "$GITHUB_OUTPUT"
187187
188-
- name: Azure login (OIDC)
188+
- name: Azure login (OIDC, tenant-only for Graph)
189189
if: ${{ steps.ids.outputs.skip == 'false' }}
190190
uses: azure/login@v2
191191
with:
192192
client-id: ${{ secrets.AZURE_CLIENT_ID }}
193193
tenant-id: ${{ secrets.AZURE_TENANT_ID }}
194-
subscription-id: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
194+
# Graph (`az ad app delete`) does not need a subscription. Passing
195+
# subscription-id together with allow-no-subscriptions still makes
196+
# the action attempt `az account set`, which fails with "No
197+
# subscriptions found" if the OIDC SP has no role on that sub.
195198
allow-no-subscriptions: true
196199

197200
- name: Delete API app registration

0 commit comments

Comments
 (0)