Skip to content

A110: Child Channel Options#529

Open
AgraVator wants to merge 13 commits into
grpc:masterfrom
AgraVator:child-channel-plugins
Open

A110: Child Channel Options#529
AgraVator wants to merge 13 commits into
grpc:masterfrom
AgraVator:child-channel-plugins

Conversation

@AgraVator
Copy link
Copy Markdown
Contributor

No description provided.

@markdroth markdroth changed the title A110: Child Channel Plugins A110: Child Channel Options Dec 24, 2025
Copy link
Copy Markdown
Member

@markdroth markdroth left a comment

Choose a reason for hiding this comment

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

The overall design looks good, but I think the doc needs work.

My main concern is that we need a much better description of the overall design in a language-agnostic way, rather than just saying that we have a different design for each language. It's true that each language has its own APIs for how per-channel options are set, but the overall goal here is still to have a way to pass a set of options to be used in child channels. The semantics for that should be the same in all languages.

Please let me know if you have any questions. Thanks!

Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
@AgraVator AgraVator requested a review from markdroth December 29, 2025 15:56
Comment thread A110-child-channel-plugins.md
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md
Copy link
Copy Markdown
Member

@markdroth markdroth left a comment

Choose a reason for hiding this comment

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

This is looking much better!

Please let me know if you have any questions. Thanks!

Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md
Comment thread A110-child-channel-plugins.md Outdated
Copy link
Copy Markdown
Member

@markdroth markdroth left a comment

Choose a reason for hiding this comment

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

This is looking really good! The only really substantive remaining comment from me is the one from my last review pass about making sure we can plumb this into channels created via an LB policy or resolver.

Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
Comment thread A110-child-channel-plugins.md Outdated
@AgraVator AgraVator requested a review from ejona86 March 25, 2026 09:24
Comment thread A110-child-channel-plugins.md

This proposal introduces a mechanism to configure "child channels"—channels created internally by gRPC components (such as xDS control plane channel). Currently, these internal channels cannot easily inherit configuration (like metric sinks and interceptors) from the user application without relying on global state. This design proposes a language-specific approach to injecting configuration: using `functional interfaces` in Java, `DialOptions` in Go, and `ChannelArgs` in Core.
Complex gRPC ecosystems often require the creation of auxiliary channels that
are not directly instantiated by the user application. The primary examples are:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In Go, the Channel passes a BuildOptions struct when creating a resolver or an LB policy. The struct is different for the two cases. But currently it contains things like credentials, dialer etc that the resolver and balancer should use if and when they are creating a Channel themselves.

We could add a new field in this struct to contain the child Channel options. As a long-term cleanup, we could deprecate some of the existing fields and move them to the newly added options field as well.

For the server, it should be simpler as the xDS-enabled server is the one that attempts to create the xDS client.

@AgraVator AgraVator requested review from easwars and markdroth May 12, 2026 06:23
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.

8 participants