Bogus serial string in subiquity install wwn-eui.*

Bug #1849322 reported by Ryan Harper on 2019-10-22
34
This bug affects 5 people
Affects Status Importance Assigned to Milestone
curtin
High
Unassigned
curtin (Ubuntu)
High
Unassigned
subiquity (Ubuntu)
Undecided
Unassigned

Bug Description

Curtin has been given (or maybe generated for subiquity) a storage config which includes a serial value with wwn prefixing the actual serial (eui-XXXXX). This results if failure to find the disk since the wwn prefix is not needed.

---
curtin: Installation started. (19.2)
start: cmd-install/stage-partitioning/builtin/cmd-block-meta: curtin command block-meta
get_path_to_storage_volume for volume disk-nvme0n1
devsync for /dev/disk/by-id/wwn-eui.01000000010000005cd2e4b23a125151
Running command ['partprobe', '/dev/disk/by-id/wwn-eui.01000000010000005cd2e4b23a125151'] with allowed return codes [0, 1] (capture=False)
Error: Could not stat device /dev/disk/by-id/wwn-eui.01000000010000005cd2e4b23a125151 - No such file or directory.
Running command ['udevadm', 'settle'] with allowed return codes [0] (capture=False)
TIMED udevadm_settle(): 0.006
Waiting on device path: /dev/disk/by-id/wwn-eui.01000000010000005cd2e4b23a125151

Related branches

Ryan Harper (raharper) wrote :

Curtin is prefixing 'wwn' for configs which specify a wwn value. The wwn value does not include the string 'wwn' and udev rules for persistent disks prefix the wwn with wwn-<WWN value).

However, for NVME, udev may export ID_WWN=eui.XXXX value, and it may well be a WWN, it does not get symlinked as a wwn-<WWN> symlink in /dev/disk/by-id at all.

Changed in subiquity (Ubuntu):
status: New → Invalid
Changed in curtin (Ubuntu):
importance: Undecided → High
status: New → In Progress

This bug is fixed with commit d55dbb04 to curtin on branch master.
To view that commit see the following URL:
https://git.launchpad.net/curtin/commit/?id=d55dbb04

Ryan Harper (raharper) on 2019-10-25
Changed in curtin:
importance: Undecided → High
status: New → Fix Committed
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package curtin - 19.2-44-g8e618b34-0ubuntu1

---------------
curtin (19.2-44-g8e618b34-0ubuntu1) focal; urgency=medium

  * New upstream snapshot.
    - t/jenkins-runner: replace $EPOCHSECONDS with 'date +%s' [Paride Legovini]
    - curthooks: skip setup_kernel_img_conf on eoan and newer (LP: #1847257)
    - block_meta: use lookup for wwn, fix fallback from wwn, serial, path
      (LP: #1849322)
    - vmtest: Adjust TestScsiBasic to use dnames to find correct disk
    - schema: Add ptable value 'unsupported' (LP: #1848535)
    - tools/xkvm: add -nographic to speed up devopt query
    - test_block_dasd: fix random_device_id to only generate valid IDs
      (LP: #1849549)
    - vmtest: update skip_if_arch message
    - Add skip_by_date to eoan ipv6 vlan test
    - storage_config: interpret value, not presence, of
      DM_MULTIPATH_DEVICE_PATH [Michael Hudson-Doyle]
    - vmtest: Add skip_by_date for test_ip_output on eoan + vlans
    - block-schema: update raid schema for preserve and metadata
    - dasd: update partition table value to 'vtoc' (LP: #1847073)
    - clear-holders: increase the level for devices with holders by one
      (LP: #1844543)
    - tests: mock timestamp used in collect-log file creation (LP: #1847138)
    - ChrootableTarget: mount /run to resolve lvm/mdadm issues which
      require it.
    - block-discover: handle multipath disks (LP: #1839915)
    - Handle partial raid on partitions (LP: #1835091)
    - install: export zpools if present in the storage-config (LP: #1838278)
    - block-schema: allow 'mac' as partition table type (LP: #1845611)
    - jenkins-runner: disable the lockfile timeout by default [Paride Legovini]
    - curthooks: use correct grub-efi package name on i386 (LP: #1845914)
    - vmtest-sync-images: remove unused imports [Paride Legovini]
    - vmtests: use file locking on the images [Paride Legovini]
    - vmtest: enable arm64 [Paride Legovini]
    - Make the vmtests/test_basic test suite run on ppc64el [Paride Legovini]
    - vmtests: separate arch and target_arch in tests [Paride Legovini]
    - vmtests: new decorator: skip_if_arch [Paride Legovini]
    - vmtests: increase the VM memory for Bionic
    - vmtests: Skip Eoan ZFS Root tests until bug fix is complete
    - Merge branch 'fix_merge_conflicts'
    - util: add support for 'tbz', 'txz' tar format types to sanitize_source
      (LP: #1843266)
    - net: ensure eni helper tools install if given netplan config
      (LP: #1834751)
    - d/control: update Depends for new probert package names
      [Dimitri John Ledkov]

 -- Ryan Harper <email address hidden> Fri, 01 Nov 2019 14:06:13 -0500

Changed in curtin (Ubuntu):
status: In Progress → Fix Released

This bug is believed to be fixed in curtin in version 19.3. If this is still a problem for you, please make a comment and set the state back to New

Thank you.

Changed in curtin:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers