[PR #10326] [MERGED] server/internal/registry: make pull send errors with Error field #13212

Closed
opened 2026-04-13 00:21:03 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/10326
Author: @bmizerany
Created: 4/17/2025
Status: Merged
Merged: 4/19/2025
Merged by: @jmorganca

Base: mainHead: bmizerany/client2makeerrorserrors


📝 Commits (1)

  • 08a8782 server/internal/registry: make pull send errors with Error field

📊 Changes

2 files changed (+9 additions, -7 deletions)

View changed files

📝 server/internal/registry/server.go (+7 -5)
📝 server/internal/registry/server_test.go (+2 -2)

📄 Description

Previously, the pull handler would send an error message in the Status field, this prevented the client from using the message as a signal to stop. In the case of the "run" command, it would follow the pull with a "show" which would print a nearly identical "not found" message for unresolved models.

Fixes #10307


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/ollama/ollama/pull/10326 **Author:** [@bmizerany](https://github.com/bmizerany) **Created:** 4/17/2025 **Status:** ✅ Merged **Merged:** 4/19/2025 **Merged by:** [@jmorganca](https://github.com/jmorganca) **Base:** `main` ← **Head:** `bmizerany/client2makeerrorserrors` --- ### 📝 Commits (1) - [`08a8782`](https://github.com/ollama/ollama/commit/08a878297b7b06ea4f21a287607664b8508b4572) server/internal/registry: make pull send errors with Error field ### 📊 Changes **2 files changed** (+9 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `server/internal/registry/server.go` (+7 -5) 📝 `server/internal/registry/server_test.go` (+2 -2) </details> ### 📄 Description Previously, the pull handler would send an error message in the Status field, this prevented the client from using the message as a signal to stop. In the case of the "run" command, it would follow the pull with a "show" which would print a nearly identical "not found" message for unresolved models. Fixes #10307 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-13 00:21:03 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/ollama#13212