Skip to content

hddtemp_smartctl: support /dev/disk/by-id/* name translation #1655

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

Merged
merged 1 commit into from
Jun 23, 2025

Conversation

nachtgeist
Copy link
Contributor

If translate_names is set, for each drive walk the symlinks created by udev below /dev/disk/by-id/ and replace the simple device name sdX by its correspondent.

That way, graphs won't jump around between devices if the simple device names are unstable between reboots.

If translate_names is set, for each drive walk the symlinks created by udev
below /dev/disk/by-id/ and replace the simple device name sdX by its
correspondent.

That way, graphs won't jump around between devices if the simple device
names are unstable between reboots.
@kenyon kenyon changed the title hddtemp_smartctl: support /dev/disk/by-id/* name translation hddtemp_smartctl: support /dev/disk/by-id/* name translation Jun 23, 2025
@kenyon kenyon merged commit e746439 into munin-monitoring:master Jun 23, 2025
1 check passed
@coveralls
Copy link

Pull Request Test Coverage Report for Build 15808204940

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 70.022%

Totals Coverage Status
Change from base Build 15764637078: 0.0%
Covered Lines: 1579
Relevant Lines: 2255

💛 - Coveralls

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

Successfully merging this pull request may close these issues.

3 participants