-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
listener: release multiple addresses listener #22097
listener: release multiple addresses listener #22097
Conversation
Signed-off-by: He Jie Xu <[email protected]>
CC @envoyproxy/api-shepherds: Your approval is needed for changes made to |
/wait |
/assign @mattklein123 |
Signed-off-by: He Jie Xu <[email protected]>
/wait |
Signed-off-by: He Jie Xu <[email protected]>
Signed-off-by: He Jie Xu <[email protected]>
/wait |
Signed-off-by: He Jie Xu <[email protected]>
/wait |
Signed-off-by: He Jie Xu <[email protected]>
/wait |
Signed-off-by: He Jie Xu <[email protected]>
Signed-off-by: He Jie Xu <[email protected]>
/wait just merge with main |
@mattklein123 This should be able to merge now. There is still some cleanup left, I will continue those cleanup after I get back from vacation next week. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome. Amazing feature!
@mattklein123 thanks for all the review and feedback! |
Signed-off-by: He Jie Xu [email protected]
Commit Message: listener: release multiple addresses listener
Additional Description:
Risk Level: low
Testing: n/a
Docs Changes: n/a
Release Notes: new feature
Platform Specific Features: n/a
Part of #11184