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.

Sign up for GitHub

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

Jump to bottom

storcli could not extract drive temperature #197

Closed
LuminatiHD opened this issue Dec 22, 2023 · 1 comment
Closed

storcli could not extract drive temperature #197

LuminatiHD opened this issue Dec 22, 2023 · 1 comment

Comments

@LuminatiHD
Copy link

The storcli script has given us errors when trying to extract the temperature of a drive. our output looks like this.

$ sudo /opt/MegaRAID/storcli/storcli64 /cALL/eALL/sALL show all J | grep "Drive Temperature"
"Drive Temperature" : " 41C (105.80 F)",
"Drive Temperature" : " 42C (107.60 F)",
"Drive Temperature" : " 41C (105.80 F)",
"Drive Temperature" : "N/A",
"Drive Temperature" : " 39C (102.20 F)",
"Drive Temperature" : " 40C (104.00 F)",
"Drive Temperature" : " 39C (102.20 F)",
"Drive Temperature" : " 26C (78.80 F)",

notice the Temperature "N/A" of one of the drives. Our server is old so it is possible that the drive is either broken or doesn't have the sensors to measure this.

I'll have up a pull request shortly trying to remedy this issue. For now I suggest making a small check if the drive temperature is even a number and skipping it if it isn't. I am open to different ideas.
LuminatiHD added a commit to LuminatiHD/node-exporter-textfile-collector-scripts that referenced this issue Dec 22, 2023
@LuminatiHD
Check if temperature is a number (see prometheus-community#197)
cbec3e0
@LuminatiHD LuminatiHD mentioned this issue Dec 22, 2023
Closed
LuminatiHD added a commit to LuminatiHD/node-exporter-textfile-collector-scripts that referenced this issue Dec 22, 2023
@LuminatiHD
Check if temperature is a number (see prometheus-community#197)
709ce32 
Signed-off-by: yoan mueller <[email protected]>
LuminatiHD added a commit to LuminatiHD/node-exporter-textfile-collector-scripts that referenced this issue Jan 11, 2024
@LuminatiHD
Check if temperature is a number
1598b83 
Closes prometheus-community#197

Signed-off-by: yoan mueller <[email protected]>
@LuminatiHD
Copy link
Author

LuminatiHD commented Feb 15, 2024 •
edited
Loading

Has been fixed here already: #201

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Check if drive temperature is a number LuminatiHD/node-exporter-textfile-collector-scripts
1 participant
@LuminatiHD