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

reverse connection: code changes for a listener that initiates reverse connections #37820

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

basundhara-c
Copy link

Commit Message: The reverse connection listener config extension defines a HTTP listener, whose sole purpose is to initiate reverse connections as described in this github issue. This PR is strictly tied with #37368.

Additional Description: This extension is relevant on the initiator envoy where reverse connections are initiated with the addition of HTTP listeners with a reverse_connection_listener_config. The reverse_connection_listener_config is defined in a proto and enumerates the clusters to which reverse connections are needed and the number of reverse connections needed to each. After this config is parsed, the connection handler creates ActiveReverseConnectionListeners which trigger the reverse connection workflow.

Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Copy link

Hi @basundhara-c, welcome and thank you for your contribution.

We will try to review your Pull Request as quickly as possible.

In the meantime, please take a look at the contribution guidelines if you have not done so already.

🐱

Caused by: #37820 was opened by basundhara-c.

see: more, trace.

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.

2 participants