intel / intel/powertelemetry

RAPL package ID validation is unnecessary and breaks collections

Open
#3 8 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
14
Forks
5
PR merge metrics
No merged PRs in 30d

Description

https://github.com/intel/powertelemetry/blob/5279ae9e8994ee1ecdb98a7ec8e2a0a20bd9e542/rapl.go#L288-L290

This code compares the package ID parsed from path and from the `name` file. However, on machines with the `psys` RAPL domain, we may have `intel-rapl:2` in the path and the name could be `package-0`, depending on how it's enumerated. Having that comparison will return an error in such case and we couldn't collect any stuff for "package-0"

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.