Skip to content

Switching to identity auth #111

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Joshua-Lester3 wants to merge 19 commits into
base: main
Choose a base branch
from
+4 −2
Open

Switching to identity auth #111

Changes from all commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
a342fa0
Logging in manually
j0shuale Jan 17, 2026
a054a6e
switch MI id to vars, as its set
j0shuale Jan 17, 2026
f5db202
Extract MI assignment
j0shuale Jan 20, 2026
03c7ab5
change user assigned arg
j0shuale Jan 20, 2026
4df4a83
remove extension
j0shuale Jan 20, 2026
b2380c2
Fix target port syntax in Azure Container App deployment
j0shuale Jan 22, 2026
3789323
Update target port for Container App and assign Managed Identity
j0shuale Jan 22, 2026
5376ad0
Add subscription and managed identity parameters for Container App de…
j0shuale Jan 22, 2026
4e24efd
Refactor Container App deployment to use dynamic identity assignment …
j0shuale Jan 22, 2026
b7d3bcd
Add Azure CLI extension for Container App deployment
j0shuale Jan 22, 2026
422cc0a
Add debug flag to Container App deployment command
j0shuale Jan 22, 2026
5e4b108
Update managed identity assignment script in deployment workflow
Joshua-Lester3 Jan 25, 2026
ff1711b
Update Azure CLI script for Container App deployment to use ACR crede…
Joshua-Lester3 Jan 26, 2026
92ce8ad
Refactor ACR login steps in deployment workflow to use secrets for cr…
Joshua-Lester3 Jan 26, 2026
78cbf72
remove commented work
Joshua-Lester3 Jan 26, 2026
5be5411
remove commented work
Joshua-Lester3 Jan 26, 2026
82288c3
Remove subscription ID from environment variables in deployment workflow
Joshua-Lester3 Jan 26, 2026
c3adae6
Merge branch 'main' of https://github.com/joshua-lester3/try
Joshua-Lester3 Feb 24, 2026
cf9d12a
chore: update Azure deployment script to use service principal creden…
Joshua-Lester3 Feb 24, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions .github/workflows/Build-Test-And-Deploy.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -237,9 +237,11 @@ jobs:
RESOURCEGROUP: ${{ vars.RESOURCEGROUP }}
CONTAINER_APP_ENVIRONMENT: ${{ vars.CONTAINER_APP_ENVIRONMENT }}
REGISTRY_URL: ${{ vars.PRODCONTAINER_REGISTRY }}
ACR_USERNAME: ${{ secrets.ESSENTIALCSHARP_ACR_USERNAME }}
ACR_PASSWORD: ${{ secrets.ESSENTIALCSHARP_ACR_PASSWORD }}
ESSENTIALCSHARP_CLIENT_SECRET: ${{ secrets.ESSENTIALCSHARP_CLIENT_SECRET }}
ESSENTIALCSHARP_CLIENT_ID: ${{ secrets.ESSENTIALCSHARP_CLIENT_ID }}
ESSENTIALCSHARP_TENANT_ID: ${{ secrets.ESSENTIALCSHARP_TENANT_ID }}
with:
inlineScript: |
az config set extension.use_dynamic_install=yes_without_prompt
az containerapp up -n $CONTAINER_APP_NAME -g $RESOURCEGROUP --image $REGISTRY_URL/try:${{ github.sha }} --environment $CONTAINER_APP_ENVIRONMENT --registry-server $REGISTRY_URL --ingress external --target-port 80 --debug
az containerapp up -n $CONTAINER_APP_NAME -g $RESOURCEGROUP --image $REGISTRY_URL/try:${{ github.sha }} --environment $CONTAINER_APP_ENVIRONMENT --registry-server $REGISTRY_URL --ingress external --target-port 80 --service-principal-client-id $ESSENTIALCSHARP_CLIENT_ID --service-principal-client-secret $ESSENTIALCSHARP_CLIENT_SECRET --service-principal-tenant-id $ESSENTIALCSHARP_TENANT_ID --debug