fix(upload): hangs indefinitely on download error instead of exiting#65
Conversation
There was a problem hiding this comment.
Pull request overview
This PR fixes Client.ModelUpload potentially hanging indefinitely when the model upload pipeline emits a non-failed error status (e.g. "error downloading") by treating such statuses as terminal failures and returning an error to the caller.
Changes:
- Treat additional pipeline error statuses (not just
"failed") as terminal failures during streamedmodelUpload. - Add unit tests covering
"error downloading"with and without an accompanying message. - Update
ModelUploadResultdocumentation to reflect failure modes beyond"failed".
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| internal/api/client.go | Ends streamed ModelUpload on error-like statuses by surfacing a terminal failure error. |
| internal/api/upload_test.go | Adds regression tests ensuring "error downloading" terminates the stream and returns an error. |
| internal/api/types.go | Updates ModelUploadResult comment to reflect error-status failure cases. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Important Review skippedAuto reviews are disabled on this repository. To trigger a review, include ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Free Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Note 🎁 Summarized by CodeRabbit FreeYour organization has reached its limit of developer seats under the Pro Plan. For new users, CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please add seats to your subscription by visiting https://app.coderabbit.ai/login.If you believe this is a mistake and have available seats, please assign one to the pull request author through the subscription management page using the link above. Comment |
For: RUN-10704