Skip to content

[CI/Build][Rust Frontend] Retire Buf schema publishing - #57046

Merged
BugenZhao merged 2 commits into
vllm-project:mainfrom
JulienDarve:jdarve/remove-buf-publishing-upstream
Sep 16, 2026
Merged

BugenZhao merged 2 commits into
vllm-project:mainfrom
JulienDarve:jdarve/remove-buf-publishing-upstream

Conversation

@JulienDarve

Copy link
Copy Markdown
Contributor

Purpose

Retire the Buf Schema Registry publication path added in #51276, following the crates.io publishing support merged in #56365.

  • Remove nightly, release, and manual Buf publication, token usage, and registry metadata.
  • Keep protobuf build/lint checks on pull requests, including forks.
  • Update the documentation to direct Rust consumers to vllm-proto and other consumers to the canonical .proto files.

Consumers following Buf labels will no longer receive schema or generated SDK updates through that channel.

Open-PR searches for Buf Schema Registry, buf publishing, 51276 in:body, and 56365 in:body found no duplicate retirement PR.

Test Plan

  • buf build rust/proto
  • buf lint rust/proto
  • pre-commit run --files .github/workflows/buf.yml rust/proto/README.md rust/proto/buf.yaml
  • git diff --check

Test Result

All checks passed with Buf CLI 1.72.0, including actionlint and Markdown lint.

Model evaluation was not run: this change affects publication and documentation only. The protobuf schemas, binding generator, and crates.io workflow are unchanged.

AI assistance: This change was implemented with assistance from OpenAI Codex.

Co-authored-by: OpenAI Codex <noreply@openai.com>
Signed-off-by: Julien Darve <jdarve@NVIDIA.com>

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@BugenZhao

Copy link
Copy Markdown
Member

[AUTOMATED] @codex review

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 15, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-15T23:39:00.460304Z 7568b2b Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Can't wait for the next one!

Reviewed commit: 7568b2b4a4

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@BugenZhao BugenZhao added the ready ONLY add when PR is ready to merge/full CI is needed label Sep 16, 2026
@BugenZhao

Copy link
Copy Markdown
Member

/ci run

@github-actions

Copy link
Copy Markdown

✅ @JulienDarve, CI is now available for this PR.

  • /ci run starts upstream CI; /amd-ci run starts AMD CI only.
  • Your branch must contain every commit currently on its upstream target branch. Merge or rebase onto the latest target branch, then rerun the command. Append --allow-stale to a run command to test an outdated branch at your own risk.
  • /ci retry retries failed jobs in the CI build for the current PR head. If the current head has no CI build, it starts a new CI build for the current head containing only jobs that failed in the latest earlier CI build for this PR.
  • /amd-ci retry retries failed jobs in AMD CI for the current PR head. Use /amd-ci run when the current head has no AMD CI build.
  • /ci cancel cancels scheduled or running CI builds for this PR branch; /amd-ci cancel does the same for AMD CI only.

@BugenZhao
BugenZhao enabled auto-merge (squash) September 16, 2026 02:06
@github-actions

Copy link
Copy Markdown

✅ Triggered Buildkite CI #89222 for commit 83ff59cf8ca9.

@BugenZhao BugenZhao left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@BugenZhao
BugenZhao merged commit fdfcbbc into vllm-project:main Sep 16, 2026
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/build ready ONLY add when PR is ready to merge/full CI is needed rust

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants