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

[FEAT] Support multiple Temperature Sensors in NVMe monitoring #727

Open
jayr0d opened this issue Nov 29, 2024 · 1 comment
Open

[FEAT] Support multiple Temperature Sensors in NVMe monitoring #727

jayr0d opened this issue Nov 29, 2024 · 1 comment

Comments

@jayr0d
Copy link

jayr0d commented Nov 29, 2024

Problem/Justification
Certain NVMe drives, such as Samsung SSD 970 EVO Plus, report Temperature, Temperature Sensor 1, and Temperature Sensor 2. Temperature is the same as Temperature Sensor 1, but Temperature Sensor 2 is the controller, which is more susceptible to heat issues.

Impact
Adding the Temperature Sensor data to the details collected will allow system administrators a better view of issues related to the controller.

User Story
Drive Details would provide data from the Temperature Sensors, if available for the drive. Separate Temperature graphs could also be created for each sensor.

@AnalogJ
Copy link
Owner

AnalogJ commented Jan 4, 2025

interesting, I wasn't aware of that.

If you want to open a PR to support this additional temp data, we process and save temp data using this function -

func (sr *scrutinyRepository) SaveSmartTemperature(ctx context.Context, wwn string, deviceProtocol string, collectorSmartData collector.SmartInfo) error {

@AnalogJ AnalogJ closed this as completed Jan 4, 2025
@AnalogJ AnalogJ reopened this Jan 4, 2025
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