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

Filter for sensor specifically remotes thresholds #13

Open
LordHepipud opened this issue Aug 16, 2023 · 2 comments
Open

Filter for sensor specifically remotes thresholds #13

LordHepipud opened this issue Aug 16, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@LordHepipud
Copy link

If you are using the plugin to collect all sensor data, you receive performance metrics for the sensors:

./check_akcp_sensorprobeXplus -h xx.xxx.xxx.xx -c community
[CRITICAL] - Device SPX+ Demo at location Hole (SPX+ F7 1.0.5233 May 12 2020 09:41:)
\_ [OK] Dual Humidity Port 1: 56.0%
\_ [OK] Dual Temperature Port 1: 23.5℃
\_ [OK] Temp Sensor Test: 23.7℃
\_ [CRITICAL] Motion Detector Port 4: Motion
\_ [OK] Icinga Demo: 0.0
|'Dual Humidity Port 1'=56%;32:66;23:69 'Dual Temperature Port 1'=23.5C;20.7:30;10.6:40 'Temp Sensor Test'=23.7C;20:30;10.5:40 'Motion Detector Port 4'=1 'Icinga Demo'=0

If you are filtering for specific sensors and ports, the thresholds are not added to the output:

./check_akcp_sensorprobeXplus -h xx.xxx.xxx.xx -c community --sensorPort 1 -m temperature
[OK] - Device SPX+ Demo at location Hole (SPX+ F7 1.0.5233 May 12 2020 09:41:)
\_ [OK] Temp Sensor Test: 23.7℃
|'Temp Sensor Test'=23.7C

While the plugin output is correct in case it is critical, the performance data thresholds are just missing.

@martialblog martialblog added the bug Something isn't working label Aug 16, 2023
@martialblog
Copy link
Member

@RincewindsHat Can you have a look at this?

@RincewindsHat
Copy link
Member

sure, if I remember correctly, the -m temperature part is not actually a filter, but queries an entirely different table on the device. I think the specialised tables do not include the threshold data.

Adding a real filter function for the main sensor table would remove this limitation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants