Add deploy-agent-aks-agentid + teardown-agent-aks-agentid skills (Entra SDK auth-sidecar on AKS) - #28
Draft
vj926 wants to merge 4 commits into
Draft
Add deploy-agent-aks-agentid + teardown-agent-aks-agentid skills (Entra SDK auth-sidecar on AKS)#28vj926 wants to merge 4 commits into
vj926 wants to merge 4 commits into
Conversation
Adds an AKS variant of the dev (Ollama) sidecar deployment alongside the existing Azure Container Apps tutorial. Mirrors the upstream layout: - deploy/azure/kubernetes-service/dev/README.md - Microsoft Learn-style walkthrough (Phase 1 Entra -> Phase 7 Verify, cost, teardown, troubleshooting, kind smoke-test appendix). - .claude/skills/deploy-agent-aks-dev/ - automated fast-path skill (orchestrator, per-phase scripts, envsubst-rendered manifests, references covering SKU sizing, workload identity, cross-tenant federation, post-deploy manual steps, troubleshooting). - .claude/skills/teardown-agent-aks-dev/ - matching teardown skill. Key architectural points: - Secretless: agent pod uses Azure Workload Identity (federated identity credential on the Blueprint app) to acquire tokens; no client secret in the cluster. - Service account 'agent-sa' in namespace 'agentid' is the federation subject (system:serviceaccount:agentid:agent-sa). - Cross-tenant supported: Entra tenant (Blueprint + Agent apps) and Azure subscription tenant (AKS + ACR) may differ; FIC trust is OIDC-URL-based. - Default path is autonomous app-only auth; user-OBO is documented as an optional add-on via port-forward in section 11.4. Validated end-to-end on an AKS Standard_D4s_v5 cluster with Ollama llama3.2:3b. Tool-calling reliability table and SKU warnings document the silent-failure modes for under-sized models. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
vj926
added a commit
to vj926/AgentID-using-EntraSDK_AKS
that referenced
this pull request
May 22, 2026
- Move manifests from sidecar/aks/manifests/ into
.claude/skills/deploy-agent-aks-dev/manifests/ (matches upstream
pattern: deployment artifacts live inside the skill).
- Add deploy/azure/kubernetes-service/dev/README.md - publication-grade
walkthrough mirroring deploy/azure/container-apps/dev structure.
- Delete sidecar/aks/ entirely. Sidecar app source is shared and lives
at sidecar/{dev,weather-api}/ upstream; per-deployment artifacts
belong in the skill. The Open-Meteo timeout/retry patches that lived
in sidecar/aks/{llm-agent,weather-api}-patched/ are preserved in git
history and can be revived as a separate small PR.
- Update scripts (smoke-test-kind.sh, 04-apply-manifests.sh) and
references (SKILL.md, non-azure-k8s.md, troubleshooting.md) to use
the new manifests location.
- Drop patched-configmap logic from 04-apply-manifests.sh since the
patched/ dirs are no longer shipped.
Upstream draft PR: microsoft/entra-agentid-samples#28
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…sent grant short-circuit - references/obo-preflight-checklist.md (new): 12 pre-flight items the agentic CLI walks before enabling OBO — Microsoft.Graph module, admin role, the four GUIDs, SP existence, redirect URI match, secure context, AllPrincipals vs Principal consent decision, assignment gating, clean browser session. Plus 4 post-run verification rows. - references/troubleshooting.md: rows for failure modes seen in real debugging — AADSTS500011 (Blueprint identifierUris empty / SP missing), silent PATCH rollback on platform-managed Blueprint, Connect-MgGraph not recognized (module missing), consentType=Principal trap masking a missing AllPrincipals grant, MSAL browser cache replay, assignment-required gating. - scripts/grant-agent-obo-consent.ps1: fix early-return bug. The pre-existing-grant short-circuit matched any User.Read grant regardless of consentType, so a Principal-typed grant could mask a missing AllPrincipals grant and the script falsely reported success. It now only short-circuits on an AllPrincipals grant and warns when only a Principal grant exists. - SKILL.md: link the new checklist from References. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…entid Aligns naming with the convention used elsewhere in the repo (<verb>-agent-<host>-<flavor>) and disambiguates from the AUID variant landing in PR microsoft#33 (deploy-agent-aks-auid). The 'dev' flavor slot is replaced with the explicit identity flavor 'agentid'. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
vj926
added a commit
to vj926/entra-agentid-samples
that referenced
this pull request
Jun 10, 2026
Aligns with the repo's <verb>-agent-<host>-<flavor> naming used by deploy-agent-aca-dev, deploy-agent-aca-aws, and (in PR microsoft#28) deploy-agent-aks-agentid. The AUID variant occupies the 'auid' flavor slot, mirroring how 'aws' differentiates the Bedrock variant of ACA. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
vj926 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
Contributor License AgreementContribution License AgreementThis Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
|
Addresses review feedback on PR microsoft#28: - Replace mixed bash/PowerShell with PowerShell-only - Rewrite teardown to remove all Entra objects created by setup - Clean up SPA redirect URIs and k8s namespace during teardown Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds an Azure Kubernetes Service variant of the dev (Ollama) sidecar sample, alongside the existing Container Apps tutorial. Same architectural pattern, AKS-native primitives.
What's in this PR
deploy/azure/kubernetes-service/dev/README.md— Microsoft Learn-style walkthrough (~700 lines).kindsmoke test (no Azure required). Appendix B: docker-compose → k8s migration notes..claude/skills/deploy-agent-aks-dev/— automated fast-path skill mirroring the layout ofdeploy-agent-aca-dev:scripts/— per-phase shell + PowerShell scripts,deploy-aks-dev.shorchestrator,smoke-test-kind.sh.manifests/— six envsubst-rendered YAMLs (namespace, ServiceAccount, weather-api, Ollama, llm-agent, LoadBalancer).references/— SKU sizing, Workload Identity primer, cross-tenant federation, post-deploy manual steps, troubleshooting, non-Azure k8s portability notes..claude/skills/teardown-agent-aks-dev/— matching teardown skill.Architecture highlights
system:serviceaccount:agentid:agent-sa, audienceapi://AzureADTokenExchange, issuer = the AKS cluster's OIDC URL.references/cross-tenant-federation.md.kubectl port-forwardto keep the browser in a secure-context for PKCE).Validation
End-to-end validated on AKS
Standard_D4s_v5with Ollamallama3.2:3b. Tool-calling reliability table (references/sku-sizing.md) and four> [!WARNING]blocks in the tutorial document the silent-failure modes for under-sized models.Conventions followed
ms.topic: tutorial), admonition style, and section ordering mirrordeploy/azure/container-apps/dev/README.md.deploy-agent-aca-dev/teardown-agent-aca-dev.sidecar/source code.Notes for reviewers
deploy-agent-aca-dev,entra-agent-id-setup) point to skills already in this repo.