Skip to content

[Brook] Moved the Core pin to sdk-rust main and regenerated the protos. - #322

Closed
moetemp wants to merge 1 commit into
moe/brook/v3/p-upstream-mainfrom
moe/brook/v3/p0-core-pin
Closed

moetemp wants to merge 1 commit into
moe/brook/v3/p-upstream-mainfrom
moe/brook/v3/p0-core-pin

Conversation

@moetemp

@moetemp moetemp commented Oct 21, 2026

Copy link
Copy Markdown
Owner

What changed?

This PR moves the sdk-core pin to sdk-rust main (0a5d6462) and regenerates the API protos for it.

Why?

The Brook v3 Core branches start at sdk-rust main, which has newer API protos than the current pin. With the pin move in its own PR, the Brook PRs above it carry no unrelated proto changes.

How did you test it?

poe build-develop, then the gen-protos steps (nexgen 0.2.8), and pytest tests/test_plugins.py tests/test_client.py.

  • Unit Tests
  • Staging
  • End to End Tests

The Brook v3 Core branches start at sdk-rust main, which has newer API protos than the current pin. Moving the pin first keeps the proto regeneration out of the Brook layers.
@moetemp

moetemp commented Oct 25, 2026

Copy link
Copy Markdown
Owner Author

Closing as stale. This layer is folded into the v5 tip, #365, and the upstream review uses a fresh series instead. Plan: https://www.notion.so/4048fc567738814990b7c796ac28fe3f

@moetemp moetemp closed this Oct 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant