Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

xds: channel should move to TF if listener resource is not an API listener on the client #8114

Open
easwars opened this issue Feb 20, 2025 · 0 comments
Labels
Area: xDS Includes everything xDS related, including LB policies used with xDS. Type: Bug

Comments

@easwars
Copy link
Contributor

easwars commented Feb 20, 2025

The xDS client currently uses the presence of the api_listener field to determine if it should perform client-side validation checks. If this field is absent, it performs server-side validation checks. The reason for this is because the xDS client does not know whether the requested listener resource is for the client-side or server-side.

When a server-side listener resource does not contain the inbound listener configuration (i.e, address, port and filter chains), the xds-enabled grpc server moves to non-serving mode (and rejects new connection attempts in this mode).

A similar check needs to be added to the client side, i.e., if a client-side listener resource does not contain the api_listener field, the channel should be put in TRANSIENT_FAILURE and RPCs must fail.

@easwars easwars added Type: Bug Area: xDS Includes everything xDS related, including LB policies used with xDS. labels Feb 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: xDS Includes everything xDS related, including LB policies used with xDS. Type: Bug
Projects
None yet
Development

No branches or pull requests

1 participant