Add MCP support for Redpanda Cloud #25802
Merged
Conversation
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
|
|
birdayz
force-pushed
the
jb/wip-mcp
branch
3 times, most recently
from
a2aa6a8 to
b3f3616
Compare
birdayz
commented
Apr 17, 2025
src/go/rpk/go.mod
Outdated
| @@ -41,6 +42,7 @@ require ( | |||
| github.com/pkg/errors v0.9.1 | |||
| github.com/prometheus/client_model v0.6.1 | |||
| github.com/prometheus/common v0.62.0 | |||
| github.com/redpanda-data/common-go/proto v0.0.0-20250416183016-0cd4089182bd | |||
There was a problem hiding this comment.
to pull *pb.mcp.go files
this is currently on a branch, i will update before merging
birdayz
commented
Apr 17, 2025
| Organization iamv1connect.OrganizationServiceClient | ||
| ResourceGroup controlplanev1connect.ResourceGroupServiceClient | ||
| Serverless controlplanev1connect.ServerlessClusterServiceClient | ||
| Region controlplanev1connect.RegionServiceClient |
There was a problem hiding this comment.
added missing clients
birdayz
commented
Apr 17, 2025
| @@ -33,6 +33,15 @@ func newAuthInterceptor(token string) connect.UnaryInterceptorFunc { | |||
| } | |||
| } | |||
|
|
|||
| func newReloadingAuthInterceptor(f func() string) connect.UnaryInterceptorFunc { | |||
There was a problem hiding this comment.
is a specialized interceptor, that does not use a static token. instead, it can call a function that provides the token (which can be refreshed in the background)
birdayz
force-pushed
the
jb/wip-mcp
branch
3 times, most recently
from
8022cf2 to
875fa5d
Compare
r-vasquez
reviewed
Apr 17, 2025
birdayz
force-pushed
the
jb/wip-mcp
branch
6 times, most recently
from
ef2c072 to
ce7a537
Compare
|
i addressed the comments, could you have a look once more @r-vasquez ? thanks! |
CI test resultstest results on build#64852
test results on build#64873
|
r-vasquez
reviewed
Apr 22, 2025
r-vasquez
previously approved these changes
Apr 22, 2025
There was a problem hiding this comment.
LGTM, thanks!
https://github.com/redpanda-data/redpanda/pull/25802/files#r2048801329 <- Let me know when you update this and I'll approve right away.
- Add command to install the MCP server. Currently, only claude desktop is supported. - Add command to serve the stdio server. MCP clients launch it as a subprocess, and interact over stdin/stdout with it. - Authentication to cloud is completely re-used from rpk.
r-vasquez
approved these changes
Apr 22, 2025
|
/backport v25.1.x |
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.
I would like to avoid introducing a plugin for this for now, we'll be fine with the release cycle of core. We're not expecting this to change very often (and want to spare all the time required to do a plugin).
Backports Required
Release Notes