Skip to content

Conversation

@richm
Copy link
Contributor

@richm richm commented Apr 8, 2025

There is a new QEMU based test which uses the qemu/kvm capability of
github action runners. This is the basis for new bootc/image mode tests
which we will be rolling out in the near future.

ansible-lint requires that the collection path is set so that the requirements
it installs are installed in the correct place.

There has been some general github action deprecation of python versions and
ubuntu versions that we have had to fix.

Remove CONTRIBUTOR from the list of users who can trigger citest.

For more information, see

@richm richm self-assigned this Apr 8, 2025
@richm richm force-pushed the changes-20250408 branch 4 times, most recently from 5684878 to 768bb12 Compare April 11, 2025 00:08
…rsions, ubuntu versions

There is a new QEMU based test which uses the qemu/kvm capability of
github action runners.  This is the basis for new bootc/image mode tests
which we will be rolling out in the near future.

ansible-lint requires that the collection path is set so that the requirements
it installs are installed in the correct place.

There has been some general github action deprecation of python versions and
ubuntu versions that we have had to fix.

Remove `CONTRIBUTOR` from the list of users who can trigger citest.

For more information, see

* linux-system-roles/.github#98
* linux-system-roles/.github#94
* linux-system-roles/.github#93
* linux-system-roles/.github#92
* linux-system-roles/.github#91

Signed-off-by: Rich Megginson <[email protected]>
@richm richm force-pushed the changes-20250408 branch from 768bb12 to 9570aef Compare April 11, 2025 14:10
@richm richm merged commit 56f3ed6 into main Apr 11, 2025
10 of 11 checks passed
@richm richm deleted the changes-20250408 branch April 11, 2025 19:16
@martinpitt
Copy link

The f42 failure reproduces locally. Journal is a bit sparse, it just says

kdumpctl[1553]: kdump: mkdumprd: failed to make kdump initrd
kdumpctl[1553]: kdump: Starting kdump: [FAILED]

I'm running sh -x kdumpctl start directly, but even that doesn't give much info. It also fails really fast, I don't think it even starts dracut. Not sure where that set +x comes from, it breaks debugging, but it's not part of /usr/bin/kdump nor /usr/lib/dracut/dracut-functions.sh.

+ dinfo 'No kdump initial ramdisk found.'
+ set +x
kdump: No kdump initial ramdisk found.
kdump: Rebuilding /boot/initramfs-6.14.0-63.fc42.x86_64kdump.img
kdump: mkdumprd: failed to make kdump initrd
kdump: Starting kdump: [FAILED]

Hitting /usr/lib/kdump/kdump-logger.sh over the head reveals that it calls

/sbin/mkdumprd -f /boot/initramfs-6.14.0-63.fc42.x86_64kdump.img 6.14.0-63.fc42.x86_64

which fails silently. This is an internal program without --help or manpage. sh -xing it is completely inscrutable.

Maybe there's a config option which it doesn't like. Original:

# grep -v ^# /etc/kdump.conf.2385.2025-04-14@05\:33\:52~ 
auto_reset_crashkernel yes
path /var/crash
core_collector makedumpfile -l --message-level 7 -d 31

From lsr:

path /var/crash
auto_reset_crashkernel yes
failure_action reboot
dracut_args --kernel-cmdline nopti

So the core_collector option is missing (but the default value is just that according to kdump.conf(5)), and the "nopti" looks a bit weird, but that value is explicitly checked in the tests, so presumably that's ok. failure_action also looks legit.

Commenting out all options except path still fails. Adding core_collector fails. Copying the original backup file /etc/kdump.conf.2385.2025-04-14@05:33:52~ fails. So I'd say it's not the config.

/proc/cmdline has crashkernel=1G-4G:192M,4G-64G:256M,64G-:512M which is presumably ok. It's what I get with kdumpctl reset-crashkernel on a clean F42 cloud image. But when rebooting after that, it fails the same way. This is completely independent from lsr.

As it happens, this broke Cockpit's tests last week as wel, and Marius already reported it to Fedora: https://bugzilla.redhat.com/show_bug.cgi?id=2357959 . So the test really just reproduces this.

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.

3 participants