Skip to content

Remove --x-localimage experimental feature#2670

Merged
markphelps merged 1 commit intomainfrom
md/remove-localimage
Feb 2, 2026
Merged

Remove --x-localimage experimental feature#2670
markphelps merged 1 commit intomainfrom
md/remove-localimage

Conversation

@michaeldwan
Copy link
Copy Markdown
Member

@michaeldwan michaeldwan commented Jan 30, 2026

Summary

Remove the --x-localimage experimental flag and all associated code that was never shipped.

--x-localimage was an experimental flag that attempted to speed up local development by mounting weights as runtime volumes and rsyncing source files instead of copying them into the image.

Changes

  • Remove --x-localimage CLI flag from build, push, predict, run, debug, and serve commands
  • Remove LocalImage field from BuildOptions and log contexts
  • Remove localImage parameter from dockerfile generators
  • Remove FillInWeightsManifestVolumes function and WeightManifest type (dead code only used by this feature)
  • Remove CogWeightsManifestLabelKey label constant
  • Remove integration tests for the feature

Testing

  • All linting passes
  • All unit tests pass

Remove the --x-localimage flag and all associated code that was never
shipped. This includes:

- CLI flag from build, push, predict, run, debug, and serve commands
- LocalImage field from BuildOptions and log contexts
- localImage parameter from dockerfile generators
- FillInWeightsManifestVolumes and WeightManifest (dead code)
- CogWeightsManifestLabelKey label constant
- Integration tests for the feature
@michaeldwan michaeldwan marked this pull request as ready for review January 31, 2026 00:00
@michaeldwan michaeldwan requested a review from a team as a code owner January 31, 2026 00:00
Copy link
Copy Markdown
Contributor

@markphelps markphelps left a comment

Choose a reason for hiding this comment

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

nice

@markphelps markphelps merged commit d8e9353 into main Feb 2, 2026
52 of 53 checks passed
@markphelps markphelps deleted the md/remove-localimage branch February 2, 2026 15:09
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