Workaround for random kernel panic in github runner #52
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
nirs
force-pushed
the
debug
branch
9 times, most recently
from
3938b24 to
2fb267c
Compare
Add new step for inspecting guest data, logging `uname -a` and /proc/cmdline. Needed to make configurable kernel command line.
Change ssh log level to ERROR to silence warnings like:
Warning: Permanently added '192.168.105.2' (ED25519) to the list of
known hosts.
Use a dict with "image" key to specify the image url. This allows adding related images such as kernel and initrd images. Use same format to return the vm disk image.
Use disk["image"] to limit the number of temporary variable, since we need to add similar code to create kernel and initrd images. Make create_image() more generic so we can reuse it for creating kernel and initrd images that do not need any conversion. Extract clone() helper to make the code nicer to work with. We will need to clone also the kernel and initrd images.
nirs
force-pushed
the
debug
branch
4 times, most recently
from
c573b33 to
8a8ffd7
Compare
Add "kernel", "kernel_parameters", and "initrd" properties to image spec. If kernel and initrd are specified the urls are downloaded to the cache and the data is cloned to the vm directory. If kernel_parameters is specified it is copied to the disk dict. If the optional disk properties are specified we use them in the qemu command. This will be used to debug the random kernel panics in the github runner. We can use these properties later for vfkit and krunkit it needed.
nirs
force-pushed
the
debug
branch
2 times, most recently
from
4b352d2 to
57f7747
Compare
To debug the random kernel panic on the github runner we need to add the
apic=debug kernel command line parameter.
[ 0.020000] Kernel panic - not syncing: IO-APIC + timer doesn't
work! Boot with apic=debug and send a report. Then try booting
with the 'noapic' option.
Trying first to boot with kernel and initrd and the default command line
parameters.
nirs
force-pushed
the
debug
branch
2 times, most recently
from
d22d5b7 to
3783795
Compare
nirs
changed the title
To get more info on the random kernel panic for the qemu bug: https://gitlab.com/qemu-project/qemu/-/issues/2832
Based on socket_vmnet fix: lima-vm/socket_vmnet#56
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add support for setting kernel command line parameters and add the workaround from socket_vment.
lima-vm/socket_vmnet#56
Also add apic=debug to get more info on that may help to resolve the qemu issue:
https://gitlab.com/qemu-project/qemu/-/issues/2832
Fixes #34