Skip to content

feat: add Azure Container Linux CVM SIG config - #9340

Open
Ankita Pareek (Ankita13-code) wants to merge 3 commits into
mainfrom
ankitapareek/acl-cvm-sig-config
Open

feat: add Azure Container Linux CVM SIG config#9340
Ankita Pareek (Ankita13-code) wants to merge 3 commits into
mainfrom
ankitapareek/acl-cvm-sig-config

Conversation

@Ankita13-code

@Ankita13-code Ankita Pareek (Ankita13-code) commented Aug 31, 2026

Copy link
Copy Markdown

What this PR does / why we need it:

Adds AgentBaker SIG configuration for the Azure Container Linux Confidential VM Gen2 distro, aks-acl-cvm-gen2, so AKS can resolve the distro to its Azure Linux gallery image.

Changes:

  • Adds the AKSACLCVMGen2 distro constant and includes it in AKSDistrosAvailableOnVHD.
  • Registers the distro in the containerd, Gen2, and Azure Container Linux availability lists.
  • Adds the SIGACLCVMGen2ImageConfigTemplate and Azure Linux SIG config-map entry.
  • Uses the generated SIG definition aclgen2CVMSpecialized: the release template generates aclgen2CVM, then ensure_sig_image_name_linux appends Specialized for CVM images.
  • Adds unit-test coverage for maintained-image registration, cloud-specific SIG resolution, ACL classification, and version-override resolution.

Validation:

  • go test ./pkg/agent/...

Related work and dependency:

Which issue(s) this PR fixes:

Fixes #

Signed-off-by: Ankita Pareek <ankitapareek@microsoft.com>
@github-actions

github-actions Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Windows Unit Test Results

  3 files   13 suites   48s ⏱️
404 tests 404 ✅ 0 💤 0 ❌
407 runs  407 ✅ 0 💤 0 ❌

Results for commit f1ab4c9.

♻️ This comment has been updated with latest results.

Signed-off-by: Ankita Pareek <ankitapareek@microsoft.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds AgentBaker SIG configuration for Azure Container Linux CVM Gen2.

Changes:

  • Registers the new distro and capability classifications.
  • Adds Azure Linux SIG resolution and tests.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
pkg/agent/datamodel/types.go Defines and enables the distro.
pkg/agent/datamodel/sig_config.go Adds classifiers and SIG mapping.
pkg/agent/datamodel/sig_config_test.go Tests SIG registration and resolution.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread pkg/agent/datamodel/sig_config.go Outdated
Comment thread pkg/agent/datamodel/sig_config.go
Signed-off-by: Ankita Pareek <ankitapareek@microsoft.com>
Copilot AI review requested due to automatic review settings September 1, 2026 11:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.

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.

2 participants