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/testiso: fix pxe-*.rootfs-appended test on s390x #4049

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nikita-dubrovskii
Copy link
Contributor

On s390x, mk-s390image is used to create a bootable pxelinux.0 image from the live-kernel, live-initramfs, and cmdline. Because the cmdline doesn't include coreos.liveiso, systemd starts the coreos-livepxe-rootfs.service, which checks for the presence of the coreos.live.rootfs_url karg or the /etc/coreos-live-rootfs file, but neither is present. At the same time, kola generates a concatenated initramfs. This PR uses that initramfs as input for mk-s390image.

Issue: coreos/fedora-coreos-tracker#1883

@nikita-dubrovskii
Copy link
Contributor Author

Until new, based on F42, cosa comes, or qemu-9.1.3 gets a backport of https://patchwork.ozlabs.org/project/qemu-devel/patch/[email protected]/, this PR should be on hold.

On s390x, `mk-s390image` is used to create a bootable `pxelinux.0`
image from the live-kernel, live-initramfs, and cmdline. Because
the cmdline doesn't include `coreos.liveiso`, systemd starts the
`coreos-livepxe-rootfs.service`, which checks for the presence of
the `coreos.live.rootfs_url` karg or the `/etc/coreos-live-rootfs`
file, but neither is present. At the same time, `kola` generates
a concatenated initramfs. This PR uses that initramfs as input
for `mk-s390image`.
@nikita-dubrovskii nikita-dubrovskii force-pushed the s390x_fix_pxe_rootfs_appended branch from 3350dea to 48ec544 Compare March 21, 2025 12:49
@dustymabe dustymabe marked this pull request as draft March 21, 2025 12:54
@dustymabe
Copy link
Member

Converted this to draft because we are blocked on F42 COSA for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants