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

[inputs.intel_powerstat] throttleTemp and temp in msrData should be represented as int64, not uint64 #11028

Closed
pduzinki opened this issue Apr 25, 2022 · 2 comments · Fixed by #11035
Labels
bug unexpected problem or unintended behavior

Comments

@pduzinki
Copy link

pduzinki commented Apr 25, 2022

Relevant telegraf.conf

n/a

Logs from Telegraf

n/a

System info

n/a

Docker

No response

Steps to reproduce

n/a

Expected behavior

throttleTemp and temp fields in msrData struct should be represented as int64, and not uint64.

Actual behavior

throttleTemp and temp fields in msrData struct are represented as int64.

Additional info

This can be concluded from the chapter 14.8.5.2 Reading the Digital Sensor of Software Developer's Manual

@pduzinki pduzinki added the bug unexpected problem or unintended behavior label Apr 25, 2022
@reimda
Copy link
Contributor

reimda commented May 17, 2022

Hi @pduzinki the linked PR has a build you can test: #11035 (comment)

Is that PR going to work for you?

@pduzinki
Copy link
Author

Hi @reimda, yes, that PR is fine!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug unexpected problem or unintended behavior
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants