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
OTel Access Logger: support additional stat prefix #34091
Conversation
CC @envoyproxy/api-shepherds: Your approval is needed for changes made to |
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.
Thanks for adding new features.
I've left API-related comments. This change will also require a release note.
Assigning to Yan as code-owner.
/assign @yanavlasov
api/envoy/extensions/access_loggers/open_telemetry/v3/logs_service.proto
Show resolved
Hide resolved
api/envoy/extensions/access_loggers/open_telemetry/v3/logs_service.proto
Outdated
Show resolved
Hide resolved
api/envoy/extensions/access_loggers/open_telemetry/v3/logs_service.proto
Outdated
Show resolved
Hide resolved
I will review once #34072 is submitted. |
add custom stats prefix Signed-off-by: Xuyang Tao <taoxuy@google.com>
Signed-off-by: Xuyang Tao <taoxuy@google.com>
Signed-off-by: Xuyang Tao <taoxuy@google.com>
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.
/lgtm api
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.
Please add a release note.
Signed-off-by: Xuyang Tao <taoxuy@google.com>
DONE |
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.
/lgtm api
As described in the API, this allows to distinguish emitted statistics between configured open telemetry access loggers in the logger chain. If empty, the default
access_logs.open_telemetry_access_log.
will be used.It is based on #34072 so please review from the 4th commit.