-
Notifications
You must be signed in to change notification settings - Fork 229
[k8s] Define semantic conventions for k8s cpu metrics #1489
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
Comments
I wonder if we could/should change Along with the following (from #1032): I know this would be a breaking change in the Collector but I want to check if we see value in doing this for consistency. |
Correcting myself after discussing this in the K8s SemConv stability SIG: We cannot have I don't have a better alternative here than what we have already defined in the Collector but if anybody has feel free to share it. |
Thinking of it again I think we could go with sth like |
Area(s)
area:k8s
Is your change request related to a problem? Please describe.
As part of the K8s SemConv stability work we need to define k8s cpu metrics that are already in use by the Opentelemetry Collector.
Related to #1032
Describe the solution you'd like
The existing metrics in use that we need to define as semantic conventions are the following:
k8s.node.cpu.usage
:✅ #1320k8s.node.cpu.utilization
: Blocked until open-telemetry/opentelemetry-collector-contrib#27885 (comment) is coveredk8s.node.cpu.time
: ✅ #1320k8s.pod.cpu.usage
: ✅ #1320k8s.pod.cpu.utilization
:Deprecated
k8s.pod.cpu.time
: ✅ #1320container.cpu.usage
: ✅ #1128container.cpu.utilization
:Deprecated
container.cpu.time
✅ #282k8s.container.cpu_limit_utilization
k8s.container.cpu_request_utilization
k8s.container.cpu.node.utilization
k8s.pod.cpu.node.utilization
k8s.pod.cpu_limit_utilization
k8s.pod.cpu_request_utilization
Describe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: