Skip to content

Conversation

@rzabarazesh
Copy link
Collaborator

@rzabarazesh rzabarazesh commented Nov 14, 2025

Purpose

Replacing the base image from devel to runtime and add tools and headers explicitly to save some space. For full context see #28643

Test Plan

CI

Test Result

Final image size for "test" target.
Before:

localhost/latest latest f65b4b07ced6 1 days ago 34.5 GB

After:

localhost/latest latest 35e251f68a6f 2 minutes ago 31.5 GB

~ 3GBs smaller ( ~9% )


Essential Elements of an Effective PR Description Checklist
  • The purpose of the PR, such as "Fix some issue (link existing issues this PR will resolve)".
  • The test plan, such as providing test command.
  • The test results, such as pasting the results comparison before and after, or e2e results
  • (Optional) The necessary documentation update, such as updating supported_models.md and examples for a new model.
  • (Optional) Release notes update. If your change is user facing, please update the release notes draft in the Google Doc.

@rzabarazesh rzabarazesh marked this pull request as ready for review November 14, 2025 14:55
@rzabarazesh rzabarazesh added the ready ONLY add when PR is ready to merge/full CI is needed label Nov 14, 2025
@mergify mergify bot added the ci/build label Nov 14, 2025
@rzabarazesh rzabarazesh force-pushed the docker-switch-to-runtime branch 2 times, most recently from 71ae72d to 438d6db Compare November 14, 2025 16:17
@rzabarazesh rzabarazesh force-pushed the docker-switch-to-runtime branch from 438d6db to 152ef38 Compare November 14, 2025 19:26
@rzabarazesh rzabarazesh changed the title [WIP] Switch to runtime image Switch to runtime image Nov 14, 2025
@rzabarazesh rzabarazesh changed the title Switch to runtime image Switch docker base image to runtime image Nov 14, 2025
@yeqcharlotte yeqcharlotte changed the title Switch docker base image to runtime image [CI/Build] Switch docker base image to runtime image Nov 14, 2025
@bbartels
Copy link
Contributor

Now that #26966 is merged, more runtime image dependencies can be culled. I have raised #29270, which seems to be the minimal set of dependencies needed for JIT compilation of dependencies to work.

Signed-off-by: Reza Barazesh <rezabarazesh@meta.com>
@rzabarazesh
Copy link
Collaborator Author

Closing as this was merged into #29270

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

Labels

ci/build ready ONLY add when PR is ready to merge/full CI is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants