Skip to content

WiP: Add internal listener #16095

Closed
lambdai wants to merge 44 commits intoenvoyproxy:mainfrom
lambdai:addlistenerontopofintlistener
Closed

WiP: Add internal listener #16095
lambdai wants to merge 44 commits intoenvoyproxy:mainfrom
lambdai:addlistenerontopofintlistener

Conversation

@lambdai
Copy link
Copy Markdown
Contributor

@lambdai lambdai commented Apr 21, 2021

Commit Message:
Add the listener for internal address.
This PR provide the add/remove functionality but the connection establishment to internal listener is not added yet.

Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

lambdai added 30 commits May 20, 2021 11:00
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
@lambdai lambdai force-pushed the addlistenerontopofintlistener branch from aba7605 to 46f830a Compare May 21, 2021 00:10
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
@lambdai lambdai force-pushed the addlistenerontopofintlistener branch from 46f830a to 015fad7 Compare May 21, 2021 07:16
lambdai added 12 commits May 21, 2021 13:59
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
@alyssawilk
Copy link
Copy Markdown
Contributor

@lambdai is this ready for review, or will you remove the draft status when it is?

@lambdai
Copy link
Copy Markdown
Contributor Author

lambdai commented Jun 2, 2021

@lambdai is this ready for review, or will you remove the draft status when it is?

@alyssawilk Thank you for watching this! This PR is too big to review so the first part should be #16763. WDTY?

@junr03
Copy link
Copy Markdown
Member

junr03 commented Jun 2, 2021

@lambdai given you are breaking this PR into smaller ones, I am going to close this one. The maintainers are trying to keep PR assignment fresh, so we want to close PRs that don't have active development.

@junr03 junr03 closed this Jun 2, 2021
@lambdai
Copy link
Copy Markdown
Contributor Author

lambdai commented Jun 2, 2021

@lambdai given you are breaking this PR into smaller ones, I am going to close this one. The maintainers are trying to keep PR assignment fresh, so we want to close PRs that don't have active development.

I see. Thank you for letting me know

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.

4 participants