Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lxd/instance/drivers/driver_qemu: force 4MB UEFI firmware in snap #12515

Merged
merged 2 commits into from
Nov 21, 2023

Conversation

mihalicyn
Copy link
Member

@mihalicyn mihalicyn commented Nov 7, 2023

Force 4MB edk2 firmware for existing VMs with 2MB firmware

mihalicyn added a commit to mihalicyn/lxd-pkg-snap that referenced this pull request Nov 7, 2023
canonical/lxd#12515

Signed-off-by: Alexander Mikhalitsyn <aleksandr.mikhalitsyn@canonical.com>
gabrielmougard
gabrielmougard previously approved these changes Nov 8, 2023
Copy link
Contributor

@gabrielmougard gabrielmougard left a comment

Choose a reason for hiding this comment

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

Only one general question on my side:

I guess that forcing a new NVRAM on existing VMs can have significant impacts ?

@mihalicyn
Copy link
Member Author

I guess that forcing a new NVRAM on existing VMs can have significant impacts ?

yes, and goal of this PR is to make it work without bothering users. Idea is to silently replace
old 2MB format NVRAM file to a new 4MB one.

MusicDin
MusicDin previously approved these changes Nov 15, 2023
Copy link
Member

@MusicDin MusicDin left a comment

Choose a reason for hiding this comment

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

LGTM

…rom setupNvram

Signed-off-by: Alexander Mikhalitsyn <aleksandr.mikhalitsyn@canonical.com>
- If VM uses 2 meg firmware, we force NVRAM cleanup and regeneration
and then make it to use 4 meg version.

- start using 4 meg firmware in the checkFeatures(...)

Signed-off-by: Alexander Mikhalitsyn <aleksandr.mikhalitsyn@canonical.com>
Copy link
Member

@MusicDin MusicDin left a comment

Choose a reason for hiding this comment

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

LGTM

mihalicyn added a commit to mihalicyn/lxd-pkg-snap that referenced this pull request Nov 21, 2023
canonical/lxd#12515

Signed-off-by: Alexander Mikhalitsyn <aleksandr.mikhalitsyn@canonical.com>
@tomponline tomponline merged commit f86e644 into canonical:main Nov 21, 2023
25 of 26 checks passed
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.

4 participants