Skip to content

[smg] update gRPC proto to match upstream changes#16764

Merged
slin1237 merged 1 commit intomainfrom
proto-update
Jan 9, 2026
Merged

[smg] update gRPC proto to match upstream changes#16764
slin1237 merged 1 commit intomainfrom
proto-update

Conversation

@slin1237
Copy link
Copy Markdown
Collaborator

@slin1237 slin1237 commented Jan 9, 2026

  • GenerateRequest: Use oneof input {tokenized, text} instead of direct field
  • GenerateResponse: Remove request_id and error variant (errors via gRPC status)
  • SamplingParams: temperature now optional, top_k/max_tokens/min_tokens/n use uint32
  • AbortRequest: Use repeated request_ids instead of single request_id + reason
  • AbortResponse: Now empty
  • EmbedResponse: Simplified to direct fields
  • GetModelInfoResponse/GetServerInfoResponse: Use uint32 for counts
  • Update Rust client code and proto_wrapper for new structures

Checklist

Review Process

  1. Ping Merge Oncalls to start the PR flow. See the PR Merge Process.
  2. Get approvals from CODEOWNERS and other reviewers.
  3. Trigger CI tests with comments (/tag-run-ci-label, /rerun-failed-ci, /tag-and-rerun-ci) or contact authorized users to do so.
  4. After green CI and required approvals, ask Merge Oncalls to merge.

- GenerateRequest: Use oneof input {tokenized, text} instead of direct field
- GenerateResponse: Remove request_id and error variant (errors via gRPC status)
- SamplingParams: temperature now optional, top_k/max_tokens/min_tokens/n use uint32
- AbortRequest: Use repeated request_ids instead of single request_id + reason
- AbortResponse: Now empty
- EmbedResponse: Simplified to direct fields
- GetModelInfoResponse/GetServerInfoResponse: Use uint32 for counts
- Update Rust client code and proto_wrapper for new structures
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@slin1237 slin1237 merged commit 1bc7aa5 into main Jan 9, 2026
58 of 60 checks passed
@slin1237 slin1237 deleted the proto-update branch January 9, 2026 00:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants