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
Improve evaluation of trigger state for "update status" mode #22
Conversation
36b84c0
to
249e11a
Compare
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.
Please adjust the README.md accordingly. (Regarding -w
& -c
which is now unnecessary to provide).
Otherwise LGTM, thx!
2e0f7cf
to
e590114
Compare
| update | Shows the current DSM version and if DSM update is available | set w/c to any int this triggers: <br> warning if available and critical <br> if other than un-/available | | ||
| update | Shows the current DSM version and if DSM update is available | if update is "Unavailable", will trigger OK <br> if update is "Available", will trigger WARNING <br> otherwise: UNKNOWN | | ||
| status | Shows model, s/n, temp and status of system, fan, cpu fan and power supply | if temp higher than w/c in °C | |
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.
e590114 adjusts the README, as you suggested.
Hi Frederic,
I believe this check to evaluate theCRITICAL
status for the "status update" sensor might never have worked. This patch fixes it.after reviewing the corresponding logic, it was considered to implement another proposal:
In this way, there will be no
CRITICAL
state on this mode at all.With kind regards,
Andreas.