Skip to content

libvirt: Skip unsupported firmware types#1

Open
Brittlejf wants to merge 1 commit intoelastx/yogafrom
yoga-24
Open

libvirt: Skip unsupported firmware types#1
Brittlejf wants to merge 1 commit intoelastx/yogafrom
yoga-24

Conversation

@Brittlejf
Copy link
Copy Markdown
Member

Ignore (1) stateless mode firmware and (2) memory device firmware which do not include a few core keys such as nvram-template. This is a temporal (and backportable) workaround until firmware detection using libvirt's internal feature is implemented by [1]

[1] https://blueprints.launchpad.net/nova/+spec/libvirt-firmware-auto-selection

Closes-Bug: #2122288
Change-Id: I99bc36fdd5df816c9ae374db71e4734fb7fc467b

(cherry picked from commit d2188b9) (cherry picked from commit 53c253d)

Ignore (1) stateless mode firmware and (2) memory device firmware which
do not include a few core keys such as nvram-template. This is
a temporal (and backportable) workaround until firmware detection using
libvirt's internal feature is implemented by [1]

[1] https://blueprints.launchpad.net/nova/+spec/libvirt-firmware-auto-selection

Closes-Bug: #2122288
Change-Id: I99bc36fdd5df816c9ae374db71e4734fb7fc467b
Signed-off-by: Takashi Kajinami <kajinamit@oss.nttdata.com>
(cherry picked from commit d2188b9)
(cherry picked from commit 53c253d)
@Brittlejf Brittlejf requested a review from a team May 8, 2026 09:24
@Brittlejf
Copy link
Copy Markdown
Member Author

Fixes errors that appear during creation of uefi/q35 machine-type instances in Nova on Yoga in Ubuntu 24

[instance: e98902c4-7f91-4011-a6b3-272dda2535ea]   File "/openstack/venvs/nova-19.1.0.dev1670/lib/python3.12/site-packages/nova/virt/libvirt/host.py", line 1826, in get_loader
[instance: e98902c4-7f91-4011-a6b3-272dda2535ea]     loader['mapping']['nvram-template']['filename'],
[instance: e98902c4-7f91-4011-a6b3-272dda2535ea]     ~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^
[instance: e98902c4-7f91-4011-a6b3-272dda2535ea] KeyError: 'nvram-template'

Copy link
Copy Markdown

@mijanss mijanss left a comment

Choose a reason for hiding this comment

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

LGTM

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