Skip to content

Add a cmdline flag to request nested virt#40

Merged
jakecorrenti merged 1 commit into
libkrun:mainfrom
slp:add-nested
Apr 7, 2025
Merged

Add a cmdline flag to request nested virt#40
jakecorrenti merged 1 commit into
libkrun:mainfrom
slp:add-nested

Conversation

@slp
Copy link
Copy Markdown
Collaborator

@slp slp commented Mar 26, 2025

Version 1.11 of libkrun introduces support for Nested Virtualization on macOS hosts. Let's add a flag to allow users to request its enablement.

Version 1.11 of libkrun introduces support for Nested
Virtualization on macOS hosts. Let's add a flag to allow users to
request its enablement.

Signed-off-by: Sergio Lopez <slp@redhat.com>
@slp
Copy link
Copy Markdown
Collaborator Author

slp commented Mar 26, 2025

Creating this one as a draft since it requires a new release of libkrun.

@slp slp marked this pull request as ready for review April 4, 2025 08:42
@slp
Copy link
Copy Markdown
Collaborator Author

slp commented Apr 4, 2025

With libkrun-1.11.0 out, we can mark this one as ready to review.

Copy link
Copy Markdown
Collaborator

@jakecorrenti jakecorrenti left a comment

Choose a reason for hiding this comment

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

LGTM. Verified on my M1 that enabling nested will fail and the option isn't required.

@jakecorrenti
Copy link
Copy Markdown
Collaborator

@tylerfanelli did you ever figure out how to shut off the verified signatures requirement on PRs?

@slp
Copy link
Copy Markdown
Collaborator Author

slp commented Apr 7, 2025

@jakecorrenti I've just disabled the requirement.

@jakecorrenti jakecorrenti merged commit 7281398 into libkrun:main Apr 7, 2025
@jakecorrenti
Copy link
Copy Markdown
Collaborator

@jakecorrenti I've just disabled the requirement.

Thanks!

@slp slp deleted the add-nested branch April 7, 2025 15:18
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.

2 participants