Skip to content

feat: Strip timestamps from base images#10331

Open
Bownairo wants to merge 2 commits into
masterfrom
eero/base-image-timestamps
Open

feat: Strip timestamps from base images#10331
Bownairo wants to merge 2 commits into
masterfrom
eero/base-image-timestamps

Conversation

@Bownairo
Copy link
Copy Markdown
Contributor

@Bownairo Bownairo commented May 27, 2026

From the fallout of #10208

The new podman seems to be able to wrap more timestamps coming from base images (back to epoch), which leads to cache pollution across the upgrade to 26.04. To avoid this, we clear all timestamps before creating cached artifacts, and make some first steps to removing timestamps from the base images entirely.

@Bownairo Bownairo added the CI_PUSH_BASE_IMAGE Publish IC-OS base images built from this branch label May 27, 2026
@github-actions github-actions Bot added the feat label May 27, 2026
@Bownairo Bownairo force-pushed the eero/base-image-timestamps branch 5 times, most recently from e6dbb53 to 2d323c4 Compare May 28, 2026 16:48
@Bownairo Bownairo force-pushed the eero/base-image-timestamps branch from 2d323c4 to b3bff06 Compare May 28, 2026 17:38
@Bownairo Bownairo marked this pull request as ready for review May 28, 2026 19:14
@Bownairo Bownairo requested review from a team as code owners May 28, 2026 19:14
Comment on lines +82 to +83
# NOTE: This does not touch base layers. As base images are currently built
# by Docker, not podman, they leak timestamps. (This doesn't really matter,
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

container-base-images.yml does not go through the same build path as build_container_base_image.py, so it's not quite as hermetic, and might give different results than local builds. The whole thing isn't reproducible anyway, which is why we use pinned containers, but it might be nice to get them on the same build path.

It's probably just as much work to remove the base images entirely, which would be a better route.

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

Labels

CI_PUSH_BASE_IMAGE Publish IC-OS base images built from this branch feat @idx @node

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants