Skip to content

Update CI image#339

Merged
Luap99 merged 1 commit into
podman-container-tools:mainfrom
mtrmac:ci-image
Sep 11, 2025
Merged

Update CI image#339
Luap99 merged 1 commit into
podman-container-tools:mainfrom
mtrmac:ci-image

Conversation

@mtrmac

@mtrmac mtrmac commented Sep 11, 2025

Copy link
Copy Markdown
Contributor

podman-container-tools/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

Luap99 commented Sep 11, 2025

Copy link
Copy Markdown
Member

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

mtrmac commented Sep 11, 2025

Copy link
Copy Markdown
Contributor Author

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

podman-container-tools/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

mtrmac commented Sep 11, 2025

Copy link
Copy Markdown
Contributor Author

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

Luap99 commented Sep 11, 2025

Copy link
Copy Markdown
Member

(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.

@Luap99 Luap99 left a comment

Copy link
Copy Markdown
Member

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 podman-container-tools:main Sep 11, 2025
37 checks passed
@mtrmac mtrmac deleted the ci-image branch September 11, 2025 20:36
@Luap99 Luap99 mentioned this pull request Jun 5, 2026
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