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

kola/qemuexec: add --netboot option #3644

Closed
wants to merge 1 commit into from
Closed

Conversation

jlebon
Copy link
Member

@jlebon jlebon commented Oct 13, 2023

For local testing, it's useful to have an easy way to netboot a system. Enable this by exposing QEMU's builtin support for TFTP serving and BOOTP option.

For iPXE, one can just directly pass the iPXE script. For PXELINUX/GRUB, you'll likely want to prepare a netboot directory with your artifacts.

Probably this should be streamlined more in the future, and also deduped more with the related bits in metal.go. But anyway, for now this is immediately useful in helping to test root on iSCSI locally (via iPXE's sanboot option).

For local testing, it's useful to have an easy way to netboot a system.
Enable this by exposing QEMU's builtin support for TFTP serving and
BOOTP option.

For iPXE, one can just directly pass the iPXE script. For PXELINUX/GRUB,
you'll likely want to prepare a netboot directory with your artifacts.

Probably this should be streamlined more in the future, and also deduped
more with the related bits in `metal.go`. But anyway, for now this is
immediately useful in helping to test root on iSCSI locally (via iPXE's
`sanboot` option).
Copy link
Contributor

@jbtrystram jbtrystram left a comment

Choose a reason for hiding this comment

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

LGTM

@jlebon
Copy link
Member Author

jlebon commented Oct 17, 2023

Merged as part of #3645.

@jlebon jlebon closed this Oct 17, 2023
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.

None yet

2 participants