Skip to content
This repository was archived by the owner on Oct 6, 2025. It is now read-only.

Comments

docker model package without --push#132

Merged
doringeman merged 7 commits intomainfrom
package-without-push
Jul 24, 2025
Merged

docker model package without --push#132
doringeman merged 7 commits intomainfrom
package-without-push

Conversation

@ekcasey
Copy link
Contributor

@ekcasey ekcasey commented Jul 24, 2025

docker model package without --push flag load resulting artifact to the model runner content store via the load endpoint (see https://github.com/docker/model-runner/pull/113/files).

Signed-off-by: Emily Casey <emily.casey@docker.com>
@ekcasey ekcasey force-pushed the package-without-push branch from f5d0b2c to 4abe9e7 Compare July 24, 2025 03:25
Signed-off-by: Emily Casey <emily.casey@docker.com>
@ekcasey ekcasey force-pushed the package-without-push branch from 4abe9e7 to c3466c3 Compare July 24, 2025 03:32
@ekcasey ekcasey marked this pull request as ready for review July 24, 2025 03:32
@ekcasey ekcasey requested review from doringeman and xenoscopic July 24, 2025 03:32
Signed-off-by: Emily Casey <emily.casey@docker.com>
@ekcasey ekcasey force-pushed the package-without-push branch from d146886 to 3074cb2 Compare July 24, 2025 04:02
Signed-off-by: Dorin Geman <dorin.geman@docker.com>
Copy link
Contributor

@xenoscopic xenoscopic left a comment

Choose a reason for hiding this comment

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

LGTM (pending final merge of docker/model-distribution#113 I guess?)

Signed-off-by: Dorin Geman <dorin.geman@docker.com>
Signed-off-by: Dorin Geman <dorin.geman@docker.com>
Signed-off-by: Dorin Geman <dorin.geman@docker.com>
@doringeman doringeman force-pushed the package-without-push branch from 60f656c to 0abe90c Compare July 24, 2025 12:53
@doringeman doringeman merged commit 67e5699 into main Jul 24, 2025
6 checks passed
@doringeman doringeman deleted the package-without-push branch July 24, 2025 12:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants