Bogus serial string in subiquity install wwn-eui.*

Bug #1849322 reported by Ryan Harper
34
This bug affects 5 people
Affects Status Importance Assigned to Milestone
curtin
Fix Released
High
Unassigned
curtin (Ubuntu)
Fix Released
High
Unassigned
subiquity (Ubuntu)
Invalid
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

Revision history for this message
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
Revision history for this message
Server Team CI bot (server-team-bot) wrote :

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)
Changed in curtin:
importance: Undecided → High
status: New → Fix Committed
Revision history for this message
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
Revision history for this message
Ryan Harper (raharper) wrote : Fixed in curtin version 19.3.

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
Revision history for this message
vmware-gos-Yuhua (yhzou) wrote :

Verified: During the installing, we are provided the option "Update to the new installer". After update installer to 19.11.1, this issue disappear.

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

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.