-
Notifications
You must be signed in to change notification settings - Fork 20
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
Check for ATA errors #102
base: master
Are you sure you want to change the base?
Check for ATA errors #102
Conversation
here's full log of another drive in similar condition:
|
What does |
default check is failing due to
|
Yep, Would that be enough or does your commit enhance this capability? (besides adding an option to skip the log check) |
|
Ah. Mixed something up, sorry. Too many projects at the same time... So yeah, your contribution is very helpful for detecting errors not showing up in any attribute. I've had a few of such cases as well in the past and would have been happy if check_smart would have detected these 👍 |
No problem. Currently the script executes following commands:
we could merge the last two if we execute |
Yeah that improvement would be #14, which would be nice, but not urgent. |
Ok, then we should add more command executions. |
@Napsty Now it's using |
Yeah, sounds good to me. It doesn't confuse with any existing SMART attribute, so this should be fine. |
The idea is to inspect ATA device logs, which are also part of
smartctl -a /dev/sda
output. If the device log contains some errors it means that some operations are already failing, non-zero value would be escalated to aWARNING
kernel log might contain errors like this:
from man page:
The actual output looks like this:
check_smart
output with this modification: