Skip to content

Conversation

@portante
Copy link
Contributor

@portante portante commented Nov 5, 2025

The make clean target now cleans up the /tmp area a bit. It leaves around the cache of the busybox image to avoid fetching it too often.

@portante portante marked this pull request as ready for review November 5, 2025 21:46
@packit-as-a-service
Copy link

Ephemeral COPR build failed. @containers/packit-build please check.

@jnovy
Copy link
Collaborator

jnovy commented Nov 24, 2025

Maybe better approach would be to just use podman for image caching without any usage of /tmp?

Benefits:

  • No /tmp files to manage at all
  • Podman's image cache is the single source of truth
  • make clean only needs to clean actual test artifacts
  • No cache directory needed

This would need: Removal of ROOTFS_CACHE_DIR and ROOTFS_CACHE_MARKER variables, cache_ubi10_rootfs() to rely solely on podman's built-in image cache and removal of redundant marker file creation and checks.

Does it make sense?

@portante
Copy link
Contributor Author

portante commented Dec 1, 2025

Maybe better approach would be to just use podman for image caching without any usage of /tmp?

I'm on board. I'll take a stab at it.

The `make clean` target now cleans up the `/tmp` area a bit.  It leverages
`podman pull --policy=newer` to avoid always pulling the UBI10 image instead of
markers in `/tmp`.

Signed-off-by: Peter Portante <peter.portante@redhat.com>
@jnovy
Copy link
Collaborator

jnovy commented Dec 2, 2025

LGTM, thanks.

@jnovy jnovy merged commit 41a76ae into containers:main Dec 2, 2025
34 of 35 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.

2 participants