ITADN

Remove conditional from Azure login step in deployment workflow

#239Closedpamelafox 创建于 2025-07-24
P
pamelafoxcommented
## Issue Description The azd deployment workflow file contains a conditional Azure login step that should be simplified. ## Current Problem The workflow currently has a conditional login step using: ```yaml if: ${{ env.AZURE_CLIENT_ID != '' }} ``` ## Recommended Fix Modify the azd deployment workflow file so that the login step does not have a conditional. There should only be a single login step that looks like: ```yaml - name: Log in with Azure (Federated Credentials) run: | azd auth login ` --client-id "$Env:AZURE_CLIENT_ID" ` --federated-credential-provider "github" ` --tenant-id "$Env:AZURE_TENANT_ID" shell: pwsh ``` Remove any additional login step that uses "Client Credentials". ## Benefits - Simplifies the workflow logic - Reduces complexity and potential failure points - Ensures consistent authentication method - Follows current best practices for Azure authentication in GitHub Actions Please review and update the affected workflow files to remove the conditional and consolidate to a single login step using Federated Credentials. ## Details **Pattern:** `if: \$\{\{ env\.AZURE_CLIENT_ID != '' \}\}` ### Found matches in 2 file(s) with 2 total match(es): **File:** `.github/workflows/azure-dev.yaml` - **Line 52:** `if: ${{ env.AZURE_CLIENT_ID != '' }}` **File:** `.github/workflows/evaluate.yaml` - **Line 108:** `if: ${{ env.AZURE_CLIENT_ID != '' }}` **Repository:** https://github.com/Azure-Samples/rag-postgres-openai-python --- *This issue was automatically created by the GitHub Repo Maintainer Agent.*
关闭于 2025-07-25 0 条评论