-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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(deps): Bump upgrade github.com/shirou/gopsutil from v3.24.4 to v4.24.9 #16023
chore(deps): Bump upgrade github.com/shirou/gopsutil from v3.24.4 to v4.24.9 #16023
Conversation
Download PR build artifacts for linux_amd64.tar.gz, darwin_arm64.tar.gz, and windows_amd64.zip. 📦 Click here to get additional PR build artifactsArtifact URLs |
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.
Thanks for the PR!
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.
Thanks @zmyzheng for your contribution! In the past we had issues when bumping gopsutils so let's hope this goes smooth this time. ;-)
Summary
Telegraf depends on gopsutil to retrieve system related metrics. However, the v3 version is no longer maintained, while the v4 version has new fixes, improves and better platform coverages.
This PR upgrades gopsutil from v3 to v4.
I have tested these plugins using gopsutil and haven't seen any issue from this upgrade so far.
Note: the only breaking changes from v3 to v4 that affects this repo is,
host/SensorsTemperatures()
moved to in the newsensors
package. This PR makes the corresponding change to the input.temp plugin to use the new package to retrieve the tempature by callingsesors.SensorsTemperatures()
insteadChecklist
Related issues
resolves #16017