Skip to content

Conversation

@bcl
Copy link
Contributor

@bcl bcl commented Jan 23, 2026

The cpio -c argument is different between Ubuntu and Fedora/CentOS/RHEL, on Ubuntu it means -H odc and on Fedora it is -H newc so let's remove any question of the type and pass '-H newc'

It adds a diagnostic 'podman image inspect' after the bootc container is build, the output should help in tracking down exactly what container is being used.

This also removes an unneeded argument (--installer-payload-ref) from the bib run, a result of copy and pasting other tests as a starting point.

supakeen
supakeen previously approved these changes Jan 23, 2026
Copy link
Member

@supakeen supakeen left a comment

Choose a reason for hiding this comment

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

lol, that's amazing

@bcl bcl marked this pull request as ready for review January 23, 2026 21:51
@bcl bcl requested a review from a team as a code owner January 23, 2026 21:51
@bcl bcl requested review from croissanne, lzap and supakeen and removed request for a team January 23, 2026 21:51
bcl added 2 commits January 26, 2026 09:17
The cpio -c argument is different between Ubuntu and Fedora/CentOS/RHEL,
on Ubuntu it means -H odc and on Fedora it is -H newc so let's remove
any question of the type and pass '-H newc'

It adds a diagnostic 'podman image inspect' after the
bootc container is build, the output should help in tracking down
exactly what container is being used.

This also removes an unneeded argument (--installer-payload-ref) from
the bib run, a result of copy and pasting other tests as a starting point.
Check the booted system to make sure it used the expected container
reference. bootc status does not work with the PXE images, but
rpm-ostree status does, so let's use that for now.
@bcl bcl force-pushed the main-fix-pxe-tests branch from 51cef6e to acdc56c Compare January 26, 2026 17:17
@supakeen supakeen merged commit 13a177b into osbuild:main Jan 26, 2026
42 of 46 checks passed
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