Skip to content

refactor: set firmware configuration during create#579

Merged
kp2099 merged 1 commit intomainfrom
fix/acpi-motherboard-layout-for-efi
Dec 15, 2025
Merged

refactor: set firmware configuration during create#579
kp2099 merged 1 commit intomainfrom
fix/acpi-motherboard-layout-for-efi

Conversation

@tenthirtyam
Copy link
Copy Markdown
Collaborator

@tenthirtyam tenthirtyam commented Dec 11, 2025

Description

Refactors to support for specifying the firmware type when creating and configuring the virtual machines. This will ensure that the APCI layout for virtual machines created with virtual hardware 20 or later with EFI (and EFI with Secure Boot) have the correct APCI motherboard layout upon creation.

Reference:

Resolved Issues

Closes #472

Rollback Plan

Revert commit.

Changes to Security Controls

None.

@tenthirtyam tenthirtyam added this to the v2.1.0 milestone Dec 11, 2025
@tenthirtyam tenthirtyam self-assigned this Dec 11, 2025
@tenthirtyam tenthirtyam added the bug Bug label Dec 11, 2025
@tenthirtyam tenthirtyam force-pushed the fix/acpi-motherboard-layout-for-efi branch 2 times, most recently from 032c55b to cfa8f79 Compare December 11, 2025 21:40
@tenthirtyam tenthirtyam changed the title fix: set acpi motherboard layout for efi refactor: set firmware configuration during create Dec 11, 2025
@tenthirtyam tenthirtyam added the refactor Refactor label Dec 11, 2025
@tenthirtyam tenthirtyam force-pushed the fix/acpi-motherboard-layout-for-efi branch 2 times, most recently from 8db430a to 96b7dce Compare December 11, 2025 21:46
@tenthirtyam tenthirtyam marked this pull request as ready for review December 11, 2025 21:46
@tenthirtyam tenthirtyam requested a review from a team as a code owner December 11, 2025 21:46
Copilot AI review requested due to automatic review settings December 11, 2025 21:46
Refactors to support for specifying the firmware type when creating and configuring the virtual machines. This will ensure that the APCI layout for virtual machines created with virtual hardware 20 or later with EFI (and EFI with Secure Boot) have the correct APCI motherboard layout upon creation.

Reference:

- https://knowledge.broadcom.com/external/article/313466/
- https://williamlam.com/2023/01/acpi-motherboard-layout-requires-efi-considerations-for-switching-vm-firmware-in-vsphere-8.html

Signed-off-by: Ryan Johnson <rya@tenthirtyam.org>
@tenthirtyam tenthirtyam force-pushed the fix/acpi-motherboard-layout-for-efi branch from 96b7dce to 34fa2bd Compare December 11, 2025 21:50

This comment was marked as resolved.

@kp2099 kp2099 merged commit 68cecba into main Dec 15, 2025
14 checks passed
@kp2099 kp2099 deleted the fix/acpi-motherboard-layout-for-efi branch December 15, 2025 09:22
@github-actions
Copy link
Copy Markdown

I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 29, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Bug refactor Refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

VMs created with EFI missing the chipset.motherboardLayout setting

3 participants