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

udp_proxy: support proxy access logging #23218

Closed
wants to merge 4 commits into from

Conversation

giantcroc
Copy link

@giantcroc giantcroc commented Sep 23, 2022

Signed-off-by: giantcroc changran.wang@intel.com

Commit Message:
Some stats like no_route and idle_timeout can't be printed by session access log,
so we need proxy-level access logging to log global stats.
Additional Description:
Risk Level: Low
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue] #23241
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@repokitteh-read-only
Copy link

CC @envoyproxy/api-shepherds: Your approval is needed for changes made to (api/envoy/|docs/root/api-docs/).
envoyproxy/api-shepherds assignee is @markdroth
CC @envoyproxy/api-watchers: FYI only for changes made to (api/envoy/|docs/root/api-docs/).

🐱

Caused by: #23218 was opened by giantcroc.

see: more, trace.

giantcroc added 4 commits September 26, 2022 10:48
Signed-off-by: giantcroc <changran.wang@intel.com>
Signed-off-by: giantcroc <changran.wang@intel.com>
Signed-off-by: giantcroc <changran.wang@intel.com>
Signed-off-by: giantcroc <changran.wang@intel.com>
@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Sep 26, 2022
@repokitteh-read-only
Copy link

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).
envoyproxy/dependency-shepherds assignee is @mattklein123

🐱

Caused by: #23218 was synchronize by giantcroc.

see: more, trace.

@giantcroc
Copy link
Author

Sorry, misoperation, change to another PR.

@giantcroc giantcroc closed this Sep 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api deps Approval required for changes to Envoy's external dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants