Last commit made on 2020-02-06
Get this branch:
git clone -b fix/detect-mbr-extended-part-types 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

b774855... by Ryan Harper on 2020-02-06

Address feedback, add docstring, handle exception, add check on pnum

ae64aa3... by Ryan Harper on 2020-02-03

Add additional unittesting for all known extended partition type values

5bda7ac... by Ryan Harper on 2020-02-03

block-meta: add unittest for find_extended_partition

Add a better error message when we detect that we fail to find an
extended partition when processing a logical partition.

cf62b2a... by Ryan Harper on 2020-01-31

block-discover: detect additional "extended" partition types in MBR

Surprise! There are additional encoding values for "extended" partitions
in MBRs. Add the 3 additional values (0x5, 0x85, 0xC5) to the mapping table
and add unittest using provided probe data.

LP: #1861251

50ffca4... by Ryan Harper on 2020-01-30

clear-holders: ensure we wipe device even if multipath enabled not not mp

On Ubuntu releases where multipath is enabled but not in-use some devices
would fail to be wiped due to logic bug. This was found in the
BcacheBasic test. Refactor this logic to ensure that we wipe either
the in-use multipath dm device or the block device value originally
supplied to the wipe function.

941c43f... by Ryan Harper on 2020-01-29

block_meta: use reliable fs_spec entries for block devices

Entries in /etc/fstab currently use UUID= in almost all cases. This
is not reliable in the face of LVM snapshots, multipath, and other
scenarios where multiple block devices may have the same UUID values.

Resolve this by following the recommendations in the Ubuntu Wiki[1]
where we will use specific udev persistent block paths which embed
a *device* uuid in the path (dm-uuid, md-uuid, etc). In addition
to modifying the fs_spec value we write, also include a comment
above each entry which includes a terse device path which may help
users recogize a disk where a UUID will not. This brings curtin
to parity with other Ubuntu installers which include:

 # /mnt was mounted on /dev/sdb1 during installation

LP: #1851899

b475db2... by Ryan Harper on 2020-01-27

multipath: handle removal of multipath partitions correctly

Curtin could handle multipath-partition removal if it was given
the device-mapper device (dm-x) however, if it was fed the
underying scsi device (e.g. sda2) this would fail when curtin
would try to clear the partition requesting exclusive access
which it could not obtain due to the device-mapper device.

This branch implements a kernel scsi device lookup to map
sdXN -> dm-N mapping which resolves the issue by allowing
curtin to wipe the dm-X device even if it has been
provided the sdXN device.

Additional Changes
 - Enable dirty_disk mode on multipath test to reproduce

LP: #1857042

a45a54c... by Ryan Harper on 2020-01-16

vmtests: skip Focal deploying Centos70 ScsiBasic

In Focal, there is a bug (#1859858) in udev which returns a
truncated value for ID_SERIAL on some scsi devices; this does
not break dname rules when the ephemeral environment matches
the target system, however if Focal deploys Bionic or in this
case Centos70, the udev in the target system produces a
different ID_SERIAL value which prevents the dname rules from
matching. Skip this test until we get a udev fix.

c0c986e... by Ryan Harper on 2020-01-15

vmtests: fix network mtu tests, separating ifupdown vs networkd

On networkd based releases, some of the existing ipv6 MTU tests
will not work, by design. The curtin ifupdown helper scripts
installed ensure that the base network interface mtu is always
set correctly; networkd requires an explicit device MTU setting
in addition to an IPV6 MTU value if they are to differ.

- Add a networkd specific mtu test config
- Add a _skip_if_not_ifupdown to avoid tests when on networkd
- Update scripts to capture additional debugging info when on

Additional changes needed:
    - Add '---' before adding the service masking to ensure
      we disable snapd during vmtest
    - Refactor journalctl commands to separately sync, flush to
      ensure we capture the compelte journal when we collect
    - allow test class to specify add_repos, system_upgrade and
      upgrade_packages, env still wins if set
    - use systemd from ppa:ddstreet/systemd until ipv6 mtu
      bug is fixed

4fa2d66... by Mike Pontillo <email address hidden> on 2020-01-10

doc: Fix kexec documentation bug.