Skip to content

Conversation

@mtrmac
Copy link
Contributor

@mtrmac mtrmac commented Sep 11, 2025

containers/skopeo#2645 has updated the expected outputs of Skopeo to match podman-sequoia-0.2.0, so we need to be running with an updated image that includes that version, not 0.1.0.

Otherwise, all tests in this repo are failing.

@containers/container-libs-maintainers PTAL, this is blocking all other work. Compare the test failure in https://api.cirrus-ci.com/v1/task/6345581385220096/logs/integration.log .

@Luap99
Copy link
Member

Luap99 commented Sep 11, 2025

Looks like storage debian tests are now failing. Note I have downgraded the debian kernel to 6.1 so I assume we likely miss a skip if not new kernel feature in the tests now because of that

@mtrmac mtrmac marked this pull request as draft September 11, 2025 19:28
@github-actions github-actions bot added the storage Related to "storage" package label Sep 11, 2025
@mtrmac
Copy link
Contributor Author

mtrmac commented Sep 11, 2025

Yes: /tmp is a tmpfs, and that supports extended attributes only since kernel 6.6.

containers/skopeo#2645 has updated
the expected outputs of Skopeo to match podman-sequoia-0.2.0,
so we need to be running with an updated image that includes that
version, not 0.1.0.

Otherwise, all tests in this repo are failing.

Use /var/tmp for graph driver tests on Debian, because /tmp
is on a tmpfs and we are using an older kernel which does not support
extended attributes on tmpfs.

This adds 20-30 extra seconds to the test runs.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
@mtrmac
Copy link
Contributor Author

mtrmac commented Sep 11, 2025

Updated to use /var/tmp as TMPDIR for Debian. That costs 20-30 extra seconds on these checks, and seems to work.

(I can’t see anything in the commit history suggesting we ever did anything different in the past — did we start relying on xattrs only recently? If not, there might be some smarter way to deal with this that I’m missing.)

@mtrmac mtrmac marked this pull request as ready for review September 11, 2025 20:16
@Luap99
Copy link
Member

Luap99 commented Sep 11, 2025

(I can’t see anything in the commit history suggesting we ever did anything different in the past — did we start relying on xattrs only recently? If not, there might be some smarter way to deal with this that I’m missing.)

/tmp wasn't a tmpfs in the past, I switched over the Ci VMs to a tmpfs like a year ago. So I guess this was never intentional one way or another.

Copy link
Member

@Luap99 Luap99 left a comment

Choose a reason for hiding this comment

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

LGTM

@Luap99 Luap99 merged commit caf99bf into containers:main Sep 11, 2025
37 checks passed
@mtrmac mtrmac deleted the ci-image branch September 11, 2025 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

storage Related to "storage" package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants