envoy
envoy copied to clipboard
quic: add listener config option to attach connection debug visitors
Commit Message: Add a way to configure a quic connection debug visitor factory that will be used to attach a debug visitor to all quic connections on the listener. Adds an interface for this new type of factory. Additional Description: Risk Level: Low Testing: Added new tests and modified existing tests in /test/common/quic. Also performed manual testing on a real machine and sent traffic to it using quic_client. Docs Changes: Update envoy.config.listener.v3.quic_config.proto inline. Release Notes: N/A Platform Specific Features: N/A [Optional Runtime guard:] [Optional Fixes #Issue] [Optional Fixes commit #PR or SHA] [Optional Deprecated:] [Optional API Considerations:]
CC @envoyproxy/api-shepherds: Your approval is needed for changes made to (api/envoy/|docs/root/api-docs/).
envoyproxy/api-shepherds assignee is @wbpcode
CC @envoyproxy/api-watchers: FYI only for changes made to (api/envoy/|docs/root/api-docs/).
/assign @RyanTheOptimist
FYI: It looks like there is a docs issue:
https://dev.azure.com/cncf/envoy/_build/results?buildId=170030&view=logs&jobId=9fd58ae0-0c50-56b1-f517-c2b9afda9c9b&j=9fd58ae0-0c50-56b1-f517-c2b9afda9c9b&t=f871f0ce-3dc6-5bd2-7916-5ec61dbe723c
/wait
/retest
Looks like an MSAN failure:
[ FAILED ] QuicHttpIntegrationTests/QuicHttpIntegrationTest.ConnectionDebugVisitor/IPv4, where GetParam() = 4-byte object <00-00 00-00>
@wbpcode friendly ping for the API review.
This looks like it's causing flakes - Will can you triage?