Repository navigation
refactor: drop the presigned URL transfers, and name missing experiments and artifacts - #57
Merged
Merged
Conversation
jwric
changed the title
Marc-AnthonyG
approved these changes
Oct 14, 2026
jwric
marked this pull request as ready for review
October 14, 2026 20:02
jwric
force-pushed
the
refactor/drop-url-transfers
branch
from
October 14, 2026 20:51
6748573 to
c10573d
Compare
jwric
force-pushed
the
refactor/drop-url-transfers
branch
from
October 14, 2026 20:59
c10573d to
8db334a
Compare
- `list_jobs`, `get_job`, `get_job_logs` (byte-offset pages) and `cancel_job` mirror the server's project job routes; responses derive Serialize. - `start_remote_job` returns the queue response, including `job_num` when the server sends it. - `ApiErrorCode::ComputeProviderJob` names a missing job.
- `start_remote_job` becomes `queue_job(namespace, project_name, compute_provider_group_name, digest, command)`, matching the route and the argument order of every other project method.
- `rust-version` is 1.97.0, the lowest toolchain that builds and tests the workspace; the client itself uses `File::lock`, stable since 1.89.
`upload_bytes_to_url` and `download_from_url` read or wrote a whole file in one request. Transfers to and from presigned URLs belong to the SDK's tracel-artifact, which streams them, uploads parts concurrently and checks downloads, so the client keeps to the Console's API.
`ApiErrorCode` gains `Experiment`, `ExperimentArtifact` and `ObjectStorageFile`, the codes the Console's experiment and artifact routes answer a 404 with, so a client can tell which was missing instead of reading them all as `Unknown`.
jwric
force-pushed
the
refactor/drop-url-transfers
branch
from
October 14, 2026 21:22
8db334a to
46a1a46
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stacked on #55.
Client::upload_bytes_to_urlandClient::download_from_urlare removed, with the transport's upload client and the timeout they used. Both read or wrote a whole file in one request. Transfers to and from presigned URLs belong to the SDK'stracel-artifact, which streams them, uploads parts concurrently, and checks downloads. The client keeps to the Console's API; nothing else used them.ApiErrorCodegainsExperiment,ExperimentArtifactandObjectStorageFile, the codes the Console's experiment and artifact routes answer a 404 with. A client can now tell which was missing instead of reading all three asUnknown.tracel-ai/tracel-cli#43 builds on both.