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

fix: allow configuring powermode check to work better with SSDs #242

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rbjorklin
Copy link

By setting --nocheck=never I can correctly scrape SMART metrics from SSDs that report a standby power state:

# hdparm -C /dev/sda

/dev/sda:
 drive state is:  standby

fixes #91

@rbjorklin rbjorklin force-pushed the introduce-powermode-check-override branch from e80b26f to b442301 Compare September 9, 2024 22:32
rbjorklin added a commit to rbjorklin/helm-charts that referenced this pull request Sep 9, 2024
rbjorklin added a commit to rbjorklin/helm-charts that referenced this pull request Sep 9, 2024
rbjorklin added a commit to rbjorklin/helm-charts that referenced this pull request Sep 9, 2024
Copy link
Contributor

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In order to avoid invalid string injection, would you please add a switch statement in main.go to validate input of the flag?

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

Successfully merging this pull request may close these issues.

Device open failed, device did not return an IDENTIFY DEVICE structure,
2 participants