-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[chore][exporter/loadbalancing] Migrate to OTel from OC metrics #33550
[chore][exporter/loadbalancing] Migrate to OTel from OC metrics #33550
Conversation
4d61bb9
to
97d172f
Compare
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.
I know the metrics and their descriptions are just being moved here, so feel free to drop my nits around wording of metric descriptions if you want to keep this PR as minimal and straightforward as possible.
I'll rebase this once the new release is out. |
Signed-off-by: Juraci Paixão Kröhling <[email protected]>
Signed-off-by: Juraci Paixão Kröhling <[email protected]>
Signed-off-by: Juraci Paixão Kröhling <[email protected]>
Signed-off-by: Juraci Paixão Kröhling <[email protected]>
25f86c5
to
89e61e6
Compare
Rebased! |
Signed-off-by: Juraci Paixão Kröhling <[email protected]>
@crobert-1, I'm merging this with two discussions that I didn't resolve but I feel that they have sufficient explanations. I'm open to considering a change to the gauge if there's a good reason for it. |
No worries, LGTM. 👍 |
This PR migrates from OpenCensus to OTel for metrics, via mdatagen.
Before:
After:
Tested with this configuration file:
And this was the test command:
Closes #33472
Signed-off-by: Juraci Paixão Kröhling [email protected]