-
Notifications
You must be signed in to change notification settings - Fork 127
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
Identify disk by ID, not system name #73
Comments
I would like this change very much. I've seen spurious false positives after device names change. This is possibly a linux issue, so it should be checked if the machine is running linux first. I'll try to fix it myself, perhaps… |
I just posted a pull request for this |
Great idea. |
See pull request #106 |
I suggest by-path, not by-id, because one can change sata hot swap disk daily to take backups. |
Agreed
…On Thu, Jul 11, 2019, 9:04 AM GabrieleV ***@***.***> wrote:
I suggest by-path, not by-id, because one can change sata hot swap disk
daily to take backups.
Same disk path, but different disk id.
Path never changes.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#73?email_source=notifications&email_token=ABK7ZNIUE6ERP4M6DLXCVQLP65K2DA5CNFSM4GHNF3F2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZXFZRI#issuecomment-510549189>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABK7ZNLTFCG6O5ZTYHCIGB3P65K2DANCNFSM4GHNF3FQ>
.
|
I still like by-id better, since if you were to accidentially reorder the disks, they stay the same. Also, I would never use hotplugging disks for backup. Backups go over the net. |
I would be nice to have a config entry to choose from by-id, by-path. |
I fully agree that this pull request is important, as the disk enumeration changes too often between reboots. Also, if there is historic data, e.g. temperature, it's difficult to track a specific device when the identifier changes. |
I noticed this is marked as an "enhancement" I suggest it's more of a bug. |
I don't know why this hasn't been pulled. The code is rather straight-forward. |
Example
/dev/disk/by-id/ata-HGST_<MODEL>_<SERIAL>
instead of
/dev/sda
which changes on reboots and such.
The text was updated successfully, but these errors were encountered: