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

Fix the initrd option in the QEMU launcher script #2239

Merged
merged 1 commit into from
Aug 16, 2024
Merged

Conversation

chewi
Copy link
Contributor

@chewi chewi commented Aug 16, 2024

Fix the initrd option in the QEMU launcher script

It was -R, but this was already taken by the read-only pflash option, so use -r instead.

How to use

Try the PXE version of the script with the -r option. You can pass a renamed copy of flatcar_production_pxe_image.cpio.gz and check the process listing to see what was passed to QEMU.

Testing done

I manually applied this change to the existing PXE script and tried the -r option. I don't think anything in CI covers this.

  • Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)
  • Inspected CI output for image differences: /boot and /usr size, packages, list files for any missing binaries, kernel modules, config files, kernel modules, etc.

It was -R, but this was already taken by the read-only pflash option, so
use -r instead.

Signed-off-by: James Le Cuirot <jlecuirot@microsoft.com>
@chewi chewi requested a review from a team August 16, 2024 16:08
@chewi chewi self-assigned this Aug 16, 2024
@chewi chewi merged commit a212694 into main Aug 16, 2024
1 check failed
@chewi chewi deleted the chewi/qemu-initrd-opt branch August 16, 2024 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging this pull request may close these issues.

3 participants