Skip to content

Add AKS ACR authentication quickstart - #505

Merged
Rebecca Calixte (Rebecca-Calixte) merged 6 commits into
masterfrom
rebecca/aks-acr-auth
Aug 28, 2026
Merged

Add AKS ACR authentication quickstart#505
Rebecca Calixte (Rebecca-Calixte) merged 6 commits into
masterfrom
rebecca/aks-acr-auth

Conversation

@Rebecca-Calixte

Copy link
Copy Markdown
Collaborator

Adds a Terraform quickstart for integrating Azure Container Registry (ACR) with Azure Kubernetes Service (AKS).

This sample:

  • Creates an AKS cluster and Azure Container Registry.
  • Grants the AcrPull role to the AKS kubelet managed identity.
  • Includes deployment, validation, and cleanup guidance in the README.
  • Migrates the previously validated sample from local storage into the Azure/terraform repository.

Work item: 568332

This README provides a step-by-step guide to authenticate Azure Kubernetes Service (AKS) with Azure Container Registry (ACR) using Terraform. It covers prerequisites, Terraform configuration, resource creation, and deployment instructions.
Copilot AI and others added 2 commits August 27, 2026 08:10
Co-authored-by: vranade-microsoft <269690592+vranade-microsoft@users.noreply.github.com>
Co-authored-by: vranade-microsoft <269690592+vranade-microsoft@users.noreply.github.com>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving the changes made by Copilot. No manual edits involved.

Added upgrade settings to the default node pool in the AKS configuration.
@Rebecca-Calixte

Copy link
Copy Markdown
Collaborator Author

E2E deployment completed successfully, but the post-apply idempotency check detected drift in the AKS default node pool’s upgrade settings. I added the provider-returned values explicitly to the default_node_pool configuration so the second Terraform plan should be empty. The E2E check will rerun with this update.

upgrade_settings {
drain_timeout_in_minutes = 0
max_surge = "10%"
node_soak_duration_in_minutes = 0

@Rebecca-Calixte
Rebecca Calixte (Rebecca-Calixte) merged commit cb6f4b9 into master Aug 28, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants