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.

Sign up for GitHub

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

Jump to bottom

better support of lm_sensors #80

Closed
bodsch opened this issue Mar 14, 2016 · 3 comments
Closed

better support of lm_sensors #80

bodsch opened this issue Mar 14, 2016 · 3 comments
Labels
feature help wanted

Comments

@bodsch
Copy link

bodsch commented Mar 14, 2016

The current state of lm_sensors support is unusable.
The last significat changes in this module are 2 years old and they include only 5 hardcoded sensors.

Why is the lm_sensors infrastructure not being used (eg. libsensors)?
@natoscott
Copy link
Member

@bodsch from looking at the git logs, I'd say its more like 10 years, or more, since development effort was put into this PMDA. I'm pretty sure it predates libsensors, and the original author is lost in the mists of time.

It would be wonderful to see it updated to use libsensors and export more metrics though, if someone wants to take on that task - AFAIK, noone is actively working on it currently.

@bodsch
Copy link
Author

bodsch commented Mar 15, 2016

Thanks for your (fast) answer! :)

@natoscott
Copy link
Member

The lmsensors PMDA was recently rewritten.

natoscott added a commit to natoscott/pcp that referenced this issue Sep 7, 2022
@natoscott
Squashed 'vendor/github.com/Nordix/hiredis-cluster/' changes from 2ae…
8059af9 
…8f460da..65fc7b81f3

65fc7b81f3 Release 0.8.1
a63aa04fed Fix async api issues (performancecopilot#107)
a439a20f4f No command retries due to CROSSSLOT
b96d43fd95 Connect using redisConnectWithOptions() (performancecopilot#103)
c47e8a9cf3 Update hiredis version in build examples
0a3b20f691 Update Makefile
4a581e91e4 Use common build warnings
08ffdd30f0 Remove usage of hiredis internal flags (performancecopilot#101)
f3091fef52 Release 0.8.0 (performancecopilot#99)
0e741c6dd8 Add Redis compatibility testing to CI (performancecopilot#97)
1aa93a0862 Add crude support for BITFIELD and BITFIELD_RO (performancecopilot#96)
295bf3c81e Add async transaction tests
6c0aecfcf1 Deprecate non-block options which have no effect (performancecopilot#89)
16ec08bb8b Accept multiple field and value arguments in HSET (performancecopilot#86)
ff76aac0f5 Timeout tests and corrections (performancecopilot#84)
7c39940fa9 Move SSL support to own library (performancecopilot#80)
573c1006f9 README updates (performancecopilot#81)
e642e42df1 Add windows and macOS builds to CI (performancecopilot#76)
5c9e294f75 tests: fix error handling in clusterclient_reconnect_async (performancecopilot#70)
4a69cb65d1 tests: add reconnect test (performancecopilot#68)
512a790dad reset cluster context errors in redisClusterAsyncFormattedCommandToNode

git-subtree-dir: vendor/github.com/Nordix/hiredis-cluster
git-subtree-split: 65fc7b81f31389b878a669710bac9d7042c2404b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature help wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@natoscott @bodsch