Remove conditional from Azure login step in deployment workflow
## 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 条评论