Skip to content

feat: add TL FIPS SIG image config and e2e#7814

Open
mxj220 wants to merge 6 commits intomainfrom
markibrahim/fips-tl-sig-config
Open

feat: add TL FIPS SIG image config and e2e#7814
mxj220 wants to merge 6 commits intomainfrom
markibrahim/fips-tl-sig-config

Conversation

@mxj220
Copy link
Contributor

@mxj220 mxj220 commented Feb 6, 2026

What this PR does / why we need it:
Adds SIG image config and E2E for TrustedLaunch FIPS 22.04 VHD
Which issue(s) this PR fixes:

Fixes #

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds support for a new Ubuntu 22.04 Gen2 FIPS + TrustedLaunch SIG image definition across the AgentBaker datamodel and e2e suite, enabling validation that this VHD can bootstrap successfully.

Changes:

  • Introduces a new Distro constant for Ubuntu 22.04 Gen2 FIPS TrustedLaunch and adds it to the “available on VHD” lists.
  • Adds a new Ubuntu SIG image config template + mapping entry for the new distro, updating SIG config unit expectations accordingly.
  • Adds an e2e VHD config entry and a new scenario test that enables Trusted Launch + FIPS 140-3 encryption capability.

Reviewed changes

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

Show a summary per file
File Description
pkg/agent/datamodel/types.go Adds new distro constant and includes it in AKSDistrosAvailableOnVHD.
pkg/agent/datamodel/sig_config.go Registers the new distro in Ubuntu/containerd/gen2 distro lists and adds a SIG image definition template + mapping entry.
pkg/agent/datamodel/sig_config_test.go Extends expected maintained SIG config map and updates expected Ubuntu SIG config count.
e2e/config/vhd.go Adds a new e2e Image entry for the Ubuntu 2204 Gen2 FIPS TL SIG image.
e2e/scenario_test.go Adds an e2e scenario validating bootstrap for the new TL+FIPS 2204 Gen2 VHD.

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.

1 participant