Skip to content

Commit 75f2959

Browse files
Merge pull request #306 from microsoft/psl-Update-SubIdreference
ci: Update subscription ID references to use secrets in deployment workflow
2 parents 384c724 + 3cd8b4c commit 75f2959

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

.github/workflows/job-deploy-linux.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,10 +91,10 @@ jobs:
9191
echo "Environment created: ${{ inputs.ENV_NAME }}"
9292
9393
echo "Setting default subscription..."
94-
azd config set defaults.subscription ${{ vars.AZURE_SUBSCRIPTION_ID }}
94+
azd config set defaults.subscription ${{ secrets.AZURE_SUBSCRIPTION_ID }}
9595
9696
# Set additional parameters
97-
azd env set AZURE_SUBSCRIPTION_ID="${{ vars.AZURE_SUBSCRIPTION_ID }}"
97+
azd env set AZURE_SUBSCRIPTION_ID="${{ secrets.AZURE_SUBSCRIPTION_ID }}"
9898
azd env set AZURE_ENV_AI_SERVICE_LOCATION="${{ inputs.AZURE_ENV_OPENAI_LOCATION }}"
9999
azd env set AZURE_LOCATION="${{ inputs.AZURE_LOCATION }}"
100100
azd env set AZURE_RESOURCE_GROUP="${{ inputs.RESOURCE_GROUP_NAME }}"

.github/workflows/job-deploy-windows.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,10 +86,10 @@ jobs:
8686
Write-Host "Environment created: ${{ inputs.ENV_NAME }}"
8787
8888
Write-Host "Setting default subscription..."
89-
azd config set defaults.subscription ${{ vars.AZURE_SUBSCRIPTION_ID }}
89+
azd config set defaults.subscription ${{ secrets.AZURE_SUBSCRIPTION_ID }}
9090
9191
# Set additional parameters
92-
azd env set AZURE_SUBSCRIPTION_ID="${{ vars.AZURE_SUBSCRIPTION_ID }}"
92+
azd env set AZURE_SUBSCRIPTION_ID="${{ secrets.AZURE_SUBSCRIPTION_ID }}"
9393
azd env set AZURE_ENV_AI_SERVICE_LOCATION="${{ inputs.AZURE_ENV_OPENAI_LOCATION }}"
9494
azd env set AZURE_LOCATION="${{ inputs.AZURE_LOCATION }}"
9595
azd env set AZURE_RESOURCE_GROUP="${{ inputs.RESOURCE_GROUP_NAME }}"

0 commit comments

Comments
 (0)