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

hddtemp returns "Success" instead of value #17

Open
screw opened this issue Dec 27, 2020 · 4 comments
Open

hddtemp returns "Success" instead of value #17

screw opened this issue Dec 27, 2020 · 4 comments

Comments

@screw
Copy link

screw commented Dec 27, 2020

One of my drives takes a long time to response and hddtemp sometimes returns "Success" instead of the actual value.

hddtemp -D /dev/sde

================= hddtemp 0.3-beta15 ==================
Model: ST2000DM001-1CH164

/dev/sde: Success

If one of the field value seems to match the temperature, be sure to read
the hddtemp man page before sending a report (section REPORT). Thanks.

Sometimes it returns correct output:

hddtemp -D /dev/sde

================= hddtemp 0.3-beta15 ==================
Model: ST2000DM001-1CH164

field(1)         = 120
field(3)         = 0
field(4)         = 122
field(5)         = 112
field(7)         = 251
field(9)         = 208
field(10)        = 0
field(12)        = 123
field(183)       = 7
field(184)       = 0
field(187)       = 23
field(188)       = 162
field(189)       = 1
field(190)       = 27
field(191)       = 0
field(192)       = 106
field(193)       = 123
field(194)       = 27
field(197)       = 56
field(198)       = 56
field(199)       = 0
field(240)       = 70
field(241)       = 154
field(242)       = 226

If one of the field value seems to match the temperature, be sure to read
the hddtemp man page before sending a report (section REPORT). Thanks.
hddtemp  /dev/sde
/dev/sde: ST2000DM001-1CH164: 27?C
@vitlav
Copy link

vitlav commented Feb 28, 2021

What is the behavior you wait for? Will you happy with an error status? Or you told about retrying the request?

@screw
Copy link
Author

screw commented Feb 28, 2021

I would love if it would return the actual value :)
Although, I ditched the failing drive so I don't care anymore ... BUT I think returning an error would be much better.
If the call really returned with success there should be a value, otherwise, there was probably an error.

Does it makes sense?

@vitlav
Copy link

vitlav commented Mar 2, 2021

I would love if it would return the actual value :)
Although, I ditched the failing drive so I don't care anymore ... BUT I think returning an error would be much better.
So you can't test hddtemp with the failing drive anymore? It is hard to reproduce the error. "Success" is a error code description.

If the call really returned with success there should be a value, otherwise, there was probably an error.
I agree. Let me try something.

@screw
Copy link
Author

screw commented Mar 15, 2021

I would love if it would return the actual value :)
Although, I ditched the failing drive so I don't care anymore ... BUT I think returning an error would be much better.
So you can't test hddtemp with the failing drive anymore? It is hard to reproduce the error. "Success" is a error code description.

Yes, I can't test it anymore.

If the call really returned with success there should be a value, otherwise, there was probably an error.
I agree. Let me try something.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants