-
Notifications
You must be signed in to change notification settings - Fork 202
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
[Enhancement] Sensors #255
Conversation
vmware_host_sensor_state vmware_host_sensor_fan vmware_host_sensor_temperature vmware_host_sensor_power_voltage vmware_host_sensor_power_current vmware_host_sensor_power_watt vmware_host_sensor_redundancy
Codecov Report
@@ Coverage Diff @@
## main #255 +/- ##
==========================================
+ Coverage 80.29% 81.80% +1.50%
==========================================
Files 4 4
Lines 873 874 +1
==========================================
+ Hits 701 715 +14
+ Misses 172 159 -13
Continue to review full report at Codecov.
|
This pull request introduces 1 alert when merging 303bf7b into dc6c807 - view on LGTM.com new alerts:
|
Memory:
|
cpu:
|
Oh strange, I got memory info on
Also I got more information on memory on my endpoint
|
runtime.healthSystemRuntime.hardwareStatusInfo.cpuStatusInfo runtime.healthSystemRuntime.hardwareStatusInfo.memoryStatusInfo
Ok i've added the 2 others metrics if you can test it |
ill test shortly and let you know. |
@M0NsTeRRR I did some testing and after merging I get this error:
|
Metrics added
vmware_host_sensor_state
vmware_host_sensor_fan
vmware_host_sensor_temperature
vmware_host_sensor_power_voltage
vmware_host_sensor_power_current
vmware_host_sensor_power_watt
vmware_host_sensor_redundancy
Some more metrics can be added but since some informations are shown with some specific models feel free to give me an extract with this script : https://gist.github.com/M0NsTeRRR/5fe8e4e44d4a6b0c1fecf2b88bd0e67c
Breaking changes
runtime.healthSystemRuntime.systemHealthInfo.numericSensorInfo
is not an alarm, i've handled it as a standard metricsruntime.healthSystemRuntime.hardwareStatusInfo.cpuStatusInfo
andruntime.healthSystemRuntime.hardwareStatusInfo.memoryStatusInfo
are removed they return me empty values all the time so I can't implement it until someone give me some datas.You can give me the data with this script : https://gist.github.com/M0NsTeRRR/5fe8e4e44d4a6b0c1fecf2b88bd0e67c
don't forget to replace line 10 with one of the two queries.
Enhancement
I think we should add in
config.py
undercollect_only
an option likesensors: False
because many peoples will use sensors from ipmi-exporter and it generates a lot of datas, what do you think ?Example
Tested on Dell R320 & Intel R1208WTTGSR