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

Mixed metrics when enable Open Telemetry sink for EG CP & DP at the same time #3361

Open
shawnh2 opened this issue May 9, 2024 · 4 comments
Assignees
Labels
area/observability Observability related issues kind/refactor

Comments

@shawnh2
Copy link
Contributor

shawnh2 commented May 9, 2024

Description:

If we enable the Open Telemetry sink for EG control-plane and data-plane at the same time, by retrieving metrics from OTel-Collector, we will get mixed metrics from both CP & DP.

Do we need to separate these metrics?

  • maybe by adding a label field for each metrcis to indicate which it comes from (DP or CP) could avoid this

/cc @envoyproxy/gateway-maintainers

Repro steps:

Environment:
latest

@shawnh2 shawnh2 added the triage label May 9, 2024
@shawnh2 shawnh2 self-assigned this May 9, 2024
@shawnh2 shawnh2 added kind/question Further information is requested area/observability Observability related issues kind/design and removed triage labels May 9, 2024
@zirain
Copy link
Contributor

zirain commented May 9, 2024

TBH, I'm not a fan of OpenTelemetry Metrics, is that anything like pod name or identity could be found?

@shawnh2 shawnh2 added kind/decision A record of a decision made by the community. and removed kind/design labels May 9, 2024
@shawnh2
Copy link
Contributor Author

shawnh2 commented May 15, 2024

the only difference is: metrics for DP shall have label prefix named envoy, like:

image

but nothing for CP.

@zirain
Copy link
Contributor

zirain commented May 15, 2024

EG should send something for identity

@shawnh2 shawnh2 added kind/refactor and removed kind/question Further information is requested kind/decision A record of a decision made by the community. labels May 16, 2024
@shawnh2
Copy link
Contributor Author

shawnh2 commented May 16, 2024

If is possible, we can add namespace label for all the metrics from CP.

xref: #3393 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/observability Observability related issues kind/refactor
Projects
None yet
Development

No branches or pull requests

2 participants