(s)ata disks do not have /dev/disk/by-path links created

Bug #1802525 reported by Dan Streetman on 2018-11-09
This bug affects 1 person
Affects Status Importance Assigned to Milestone
systemd (Ubuntu)
Dan Streetman

Bug Description


udev (in trusty) does not create symlinks in /dev/disk/by-path/ for any disks that are connected via sata.

[test case]

Install trusty onto a system with sata drives, and check /dev/disk/by-path/ for symlink(s) to those sata drive(s).

[regression potential]

the (s)ata by-path symlinks were removed due to apparent possibility of non-uniqueness:

however that was *long* ago and it has been added back in:

if there is still a possibility for non-uniqueness, then /dev/disk/by-path/ symlink(s) might be created that are non-deterministic as far as the actual disk/path they point to.

[other info]

this is 'fixed' (the by-path symlinks are created for ata disks) in Xenial and later systemd/udev.

Also bug 1611945 indicates that sata devices connected via PMP (port multiplier device) may still have incorrect by-path symlinks created, even upstream. However, since Xenial and later currently do have udev code that creates ata by-path symlinks, that is a separate issue and not a reason for keeping ata by-path symlinks about of trusty udev.

Dan Streetman (ddstreet) on 2018-11-09
Changed in systemd (Ubuntu Trusty):
status: New → In Progress
importance: Undecided → Low
assignee: nobody → Dan Streetman (ddstreet)
Changed in systemd (Ubuntu):
status: New → Fix Released
Dan Streetman (ddstreet) on 2018-11-09
description: updated
description: updated

Hello Dan, or anyone else affected,

Accepted systemd into trusty-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/systemd/204-5ubuntu20.29 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-trusty to verification-done-trusty. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-trusty. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in systemd (Ubuntu Trusty):
status: In Progress → Fix Committed
tags: added: verification-needed verification-needed-trusty
Dan Streetman (ddstreet) wrote :

ubuntu@dwalin:~$ dpkg -l | grep ' udev '
ii udev 204-5ubuntu20.28 amd64 /dev/ and hotplug management daemon
ubuntu@dwalin:~$ ls -l /dev/disk/by-path/
ls: cannot access /dev/disk/by-path/: No such file or directory

ubuntu@dwalin:~$ dpkg -l | grep ' udev '
ii udev 204-5ubuntu20.29 amd64 /dev/ and hotplug management daemon
ubuntu@dwalin:~$ ls -l /dev/disk/by-path/
total 0
lrwxrwxrwx 1 root root 9 Nov 29 21:43 pci-0000:00:1f.2-ata-1 -> ../../sda
lrwxrwxrwx 1 root root 10 Nov 29 21:43 pci-0000:00:1f.2-ata-1-part1 -> ../../sda1

tags: added: verification-done verification-done-trusty
removed: verification-needed verification-needed-trusty
Dan Streetman (ddstreet) wrote :

the linux-* autopkgtest failures are known and expected (bug 1723223) and can be ignored.

Launchpad Janitor (janitor) wrote :

This bug was fixed in the package systemd - 204-5ubuntu20.29

systemd (204-5ubuntu20.29) trusty; urgency=medium

  * d/p/0001-udev-build-by-path-identifiers-for-ATA-devices.patch
    create /dev/disk/by-path/ symlinks for (s)ata disks
    (LP: #1802525)

 -- Dan Streetman <email address hidden> Fri, 09 Nov 2018 10:16:27 -0500

Changed in systemd (Ubuntu Trusty):
status: Fix Committed → Fix Released

The verification of the Stable Release Update for systemd has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers