-
Notifications
You must be signed in to change notification settings - Fork 422
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
[kubernetes OTEL] Add kubernetes OTEL package #11137
[kubernetes OTEL] Add kubernetes OTEL package #11137
Conversation
Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
@@ -0,0 +1,3 @@ | |||
- text: OTEL |
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.
Maybe to have a Kubernetes tag as well?
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.
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.
TODO: rename OTEL -> OpenTelemetry
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.
done 531e6f8
Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
Co-authored-by: Andrew Gizas <andreas.gkizas@elastic.co>
Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
Quality Gate passedIssues Measures |
💚 Build Succeeded
History
|
Package kubernetes_otel - 0.0.1 containing this change is available at https://epr.elastic.co/search?package=kubernetes_otel |
"query": "\"metrics.k8s.node.cpu.usage\": *" | ||
}, | ||
"isBucketed": false, | ||
"label": "CPU usage Pct", |
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.
Is this dashboard finalized or just a work-in-progress/placeholder?
Just spotted that the specific Usage PCT
label seems incorrect/confusing as noted at #10910 (comment). Is there any aggregation that calculates a ratio out of the usage metric?
Didn't check for other inconsistencies in that PR.
@tetianakravchenko Does this PR replace the one in #10910 ? If so, can we please make sure the comments @ChrsMark left there are considered here? Maybe in a followup PR? |
@AlexanderWert @ChrsMark yes, it is planned for the followup PR, the whole list is in https://github.com/elastic/opentelemetry-dev/issues/454 |
Proposed commit message
Please explain:
Checklist
changelog.yml
file.Author's Checklist
How to test this PR locally
Related issues
Screenshots
Tags: