Last commit made on 2020-05-06
Get this branch:
git clone -b fix/block-discover-skip-invalid-wwn-serial https://git.launchpad.net/~raharper/curtin
Only Ryan Harper can upload to this branch. If you are Ryan Harper please log in for upload directions.

Branch merges

Branch information


Recent commits

9f8a057... by Ryan Harper on 2020-05-06

block-discover: ignore invalid id_serial/id_wwn values

When obtaining unique ids for identifying a block storage
device, block-discover looks for ID_WWN and ID_SERIAL values
and will use them in the rendered storage config. In some cases
the value of these attributes may be invalid. For example, some
consumer disks might have an invalid WWN number:


Curtin will also ingnore empty values, like

Excluded ids will not be emitted in the final storage config.

LP: #1876848

9d85462... by Ryan Harper on 2020-04-28

vmtest: rsync don't cross filesystem boundaries when copying

In some scenarios /var/lib/lxcfs is mounted with kernel bits
and rsync complains; instead we now copy using the --one-file-system
flag which avoids crossing these boundaries.

LP: #1873909

9943398... by Ryan Harper on 2020-04-27

vmtest: basic/basic_scsi adjust collect/tests for unstable device names

- For virtio-disks (TestBasic) use dname, except on centos66, use the byid
  serial name.
- For scsi-disks (TestScsiBasic) use wwn everywhere

LP: #1874100

07e9ed2... by Ryan Harper on 2020-04-23

Add unittests for partition_handler calc_[dm]_part_info and kpartx paths

ec598b0... by Ryan Harper on 2020-04-22

multipath: attempt to enforce /dev/mapper/mpath files are symlinks

Multipath device scanning and bring up variablity may result in
the creation of /dev/mapper/mpath* files that are block devices
instead of a symlink to the device mapper device (/dev/dm-0).

When these block files are present, this breaks assumptions in
curtin used to look up details in sysfs for a mapper device.

d39b9af... by Ryan Harper on 2020-04-22

block-meta: device mapper partitions may be block devices not links

In some cases the multipath library may create a block device which
is not a symbolic link to the device-mapper dev[1]. Curtin expects the
symlink. Remove the non-symlink file and allow kpartx to create it.

1. https://bugzilla.redhat.com/show_bug.cgi?id=869253

2f22690... by Ɓukasz Zemczak on 2020-04-22

Default to dm_name being id if empty earlier in dm_crypt_handler()

In other parts of the code, when deciding the path for dmcrypt_dev,
we default to 'id' when 'dm_name' is not defined. We ideally want
that to happen in all the cases. The defaulting has been there
already but a bit too late.

LP: #1874243

bcf890b... by Chad Smith on 2020-04-22

storage: correct declared schema draft version for storage schema

python3-jsonschema 3.2 (Ubuntu Focal) pays attention to the $schema
version declared in the provided schema when choosing validators.

Curtin now sets that appropriate schema draft version as 4 instead of
7 to give us better schema errors.

0ab8c80... by Dan Watkins on 2020-04-22

test_clear_holders: add missing zfs mock

0cd6569... by Ryan Harper on 2020-04-22

Mock out zfs_supported to prevent attempting to load kernel modules