Skip to content

Add securityProfile support for TrustedLaunch VMs#2020

Open
buraksekili wants to merge 5 commits into
mainfrom
fix/2013/azure-trusted-launch-security-profile
Open

Add securityProfile support for TrustedLaunch VMs#2020
buraksekili wants to merge 5 commits into
mainfrom
fix/2013/azure-trusted-launch-security-profile

Conversation

@buraksekili
Copy link
Copy Markdown
Contributor

What this PR does / why we need it:
Azure enforces TrustedLaunch on Gen2 VMs using Compute Gallery images, causing VM creation to fail with a 400 BadRequest. This PR adds a securityProfile field to the Azure cloud provider spec so users can set securityType to TrustedLaunch (with optional secureBootEnabled and vTpmEnabled) or Standard (to opt out of auto-enforced TrustedLaunch on subscriptions where Azure enables it by default).

Which issue(s) this PR fixes:
Fixes #2013

What type of PR is this?
/kind feature

Special notes for your reviewer:

Does this PR introduce a user-facing change? Then add your Release Note here:

Azure VMs now support setting `securityProfile` with `securityType` TrustedLaunch or Standard in the cloud provider spec, enabling the use of Compute Gallery images that require TrustedLaunch.

Documentation:

NONE

Azure enforces TrustedLaunch on Gen2 VMs using gallery images,
causing VM creation to fail with a 400 BadRequest. Add a
`securityProfile` field to the Azure provider spec so users can
set `securityType` to TrustedLaunch (with optional secure boot
and vTPM) or Standard (to opt out).

Fixes #2013

Signed-off-by: Burak Sekili <32663655+buraksekili@users.noreply.github.com>
@kubermatic-bot kubermatic-bot added kind/feature Categorizes issue or PR as related to a new feature. release-note Denotes a PR that will be considered when it comes time to generate release notes. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. docs/none Denotes a PR that doesn't need documentation (changes). dco-signoff: yes Denotes that all commits in the pull request have the valid DCO signoff message. labels May 11, 2026
@kubermatic-bot
Copy link
Copy Markdown
Contributor

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@kubermatic-bot kubermatic-bot added the sig/cluster-management Denotes a PR or issue as being assigned to SIG Cluster Management. label May 11, 2026
@kubermatic-bot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign ahmedwaleedmalik for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubermatic-bot kubermatic-bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label May 11, 2026
@buraksekili
Copy link
Copy Markdown
Contributor Author

/hold still working and need to test

@kubermatic-bot kubermatic-bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 11, 2026
…ween

provider and tests.

Signed-off-by: Burak Sekili <32663655+buraksekili@users.noreply.github.com>
Signed-off-by: Burak Sekili <32663655+buraksekili@users.noreply.github.com>
Signed-off-by: Burak Sekili <32663655+buraksekili@users.noreply.github.com>
replace repeated "Standard_D2s_v3" and "Standard_A2" strings with
`testVMSizeGen2` and `testVMSizeGen1` constants.

Signed-off-by: Burak Sekili <32663655+buraksekili@users.noreply.github.com>
@buraksekili buraksekili marked this pull request as ready for review May 12, 2026 14:11
@kubermatic-bot kubermatic-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 12, 2026
@kubermatic-bot
Copy link
Copy Markdown
Contributor

@buraksekili: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
pull-machine-controller-e2e-azure-custom-image-reference 45dbe38 link true /test pull-machine-controller-e2e-azure-custom-image-reference
pull-machine-controller-e2e-azure 45dbe38 link true /test pull-machine-controller-e2e-azure

Full PR test history

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dco-signoff: yes Denotes that all commits in the pull request have the valid DCO signoff message. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. docs/none Denotes a PR that doesn't need documentation (changes). kind/feature Categorizes issue or PR as related to a new feature. release-note Denotes a PR that will be considered when it comes time to generate release notes. sig/cluster-management Denotes a PR or issue as being assigned to SIG Cluster Management. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Azure VMs - Support 'TrustedLaunch' security type

2 participants