~sdhd/curtin:master

Last commit made on 2019-10-27
Get this branch:
git clone -b master https://git.launchpad.net/~sdhd/curtin
Only Sascha Dewald can upload to this branch. If you are Sascha Dewald please log in for upload directions.

Branch merges

Branch information

Name:
master
Repository:
lp:~sdhd/curtin

Recent commits

171b5b2... by Sascha Dewald

remove test_lookup_disk_wwn, because with pyudev it is duplicate

1fbf7d8... by Ryan Harper

curthooks: skip setup_kernel_img_conf on eoan and newer

In Eoan, link_in_boot is the default so curtin no longer needs
to generate a kernel-img.conf controlling link_in_boot.

LP: #1847257

d55dbb0... by Ryan Harper

block_meta: use lookup for wwn, fix fallback from wwn, serial, path

NVMe devices may provide a ID_WWN value from udev, but the NVMe rules
do not create a by-id/wwn-$ID_WWN symlink which breaks behavior with
other ID_WWN devices resulting in curtin not finding certain NVMe
devices. Resolve this issue buy using block.lookup_disk on WWN values
as well as serial.

Additional fixes to block_meta.get_path_to_storage_volume:
- get_path_to_storage_volume will try wwn, serial, and ultimately
  path values in order to find the specified disk
- Add os.path.exists check to proposed constructed values, if the
  path is not valid, we continue to use different keys if possible.
- Updated the Exception message to indicate the volume wasn't found
  and emit the config it used.

LP: #1849322

9da90cb... by Ryan Harper

vmtest: Adjust TestScsiBasic to use dnames to find correct disk

The TestScsiBasic test has some assumptions on which SCSI device
will have expected content. On PPC64, the PCI device enumeration
is not as stable as it is on x86 resulting in test failures. This
patch resolves this by leveraging the dname values and looking up
the correct kernel name for a specific target dname.

f4f329c... by Ryan Harper

schema: Add ptable value 'unsupported'

Probert output may include values of partition table 'label'
that curtin does not support. This triggers schema validation
errors for common values, like 'mac' or 'PMBR' and likely
other valid table names but are not something that curtin
can create. Instead of adding more values to the schema, mark
such table types as 'unsupported' allowing the existing table
and partitions to be re-used.

This branch also modifies block_meta disk_handler to not
attempt to validate a preserved partition table if it marked
as 'unsupported'.

LP: #1848535

9df1f86... by Ryan Harper

tools/xkvm: add -nographic to speed up devopt query

On systems which include qemu graphical components the time
needed to invoke qemu for querying if an string is a device
option can take many seconds, especially over remote connections.
Adding the '-nographic' parameter prevents qemu from loading
and launching additional resources that are not needed to determine
the devopt query answer.

eba4715... by Dan Watkins

test_block_dasd: fix random_device_id to only generate valid IDs

Previously we could generate an invalid one iff `random.randint(1,
0x10000)` returned 0x10000.

LP: #1849549

0e4dc57... by Ryan Harper

vmtest: update skip_if_arch message

Currently the skip message reads like this:

    SKIP: skip due to ppc64el=ppc64el

With this change we'll see:

    SKIP: skip due to current arch=ppc64el

0390735... by Ryan Harper

Add skip_by_date to eoan ipv6 vlan test

70fa30b... by Michael Hudson-Doyle

storage_config: interpret value, not presence, of DM_MULTIPATH_DEVICE_PATH

Curtin assumes any block device that has the DM_MULTIPATH_DEVICE_PATH
key set in udev is a block device, but that's not correct. Since
multipath-tools 0.7.7 (according to
https://www.spinics.net/lists/dm-devel/msg35965.html), _every_ block
device that multipathd looks at gets that key set, but it is set to "1"
for things that are part of a multipathed device and "0" for things that
are not. So fix the checks in curtin to follow that.