~mwhudson/curtin:pare-down-dasdview-parsing

Branch merges

Propose for merging

Merged into curtin:master

Server Team CI bot: Approve (continuous-integration) on 2020-12-11

Paride Legovini: Approve on 2020-12-10

Diff: 717 lines (+116/-400)

2 files modified

curtin/block/dasd.py (+43/-232)
tests/unittests/test_block_dasd.py (+73/-168)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

e9b3d01... by Michael Hudson-Doyle on 2020-11-25

add a comment explaining how we go from device_id to kname

b070e5b... by Michael Hudson-Doyle on 2020-11-18

pare down parsing of dasdview output to just extracting format type

8818498... by Michael Hudson-Doyle on 2020-11-19

fix verification of vtoc partitions

Currently curtin verifies the size and flags of a partition by
using sfdisk to inspect it and checking the results. But that
doesn't work on a dasd's vtoc partition table, so this branch
renames parition_verify to parition_verify_sfdisk, adds
parition_verify_fdasd and calls the latter when ptable ==
'vtoc'. The branch does a little more than is strictly necessary,
perhaps, but the other stuff will come in useful soon, I promise
:)

LP: #1899471

beb12f0... by Michael Hudson-Doyle on 2020-11-19

create an empty vtoc in disk_handler

Another merge proposal I have uses fdasd to get the block and track size
when partitioning a disk. But the wiping disk_handler can do can do
enough damage to stop fdasd working at all. Putting an empty vtoc onto
the disk allows fdasd to report the sizes when needed.

2798a68... by Michael Hudson-Doyle on 2020-11-18

remove unused parameters from dasd code

The functions that parse dasdinfo and dasdview output take
parameters strict and rawoutput that are only passed by tests.
This removes them and the tests.

af7852f... by Michael Hudson-Doyle on 2020-11-18

remove support for calling get_path_to_storage_volume on a dasd action

there is no code path that will do this today, and this implementation
will not work with some changes I want to make to support dasds passed
via virtio.

e19f3ff... by Ryan Harper on 2020-11-13

clear-holders: fix identification of multipath partitions

identify_partition calls multipath.is_mpath_partition using at sysfs
path to the device but is_mpath_partition expects a /dev/XXX path.
This prevented clear-holders from identifying multipath partitions which
need to be handled differently than regular partitions (or disks).

LP: #1900900

1f5755c... by Dan Watkins on 2020-10-20

vmtests: remove skip_by_dates for now-fixed bcache issue

59ea037... by Paride Legovini on 2020-10-19

debian/rules: drop PKG_VERSION and UPSTREAM_VERSION

The PKG_VERSION variable in debian/rules is not used and the command
generating it doesn't work, causing a 'Bad substitution' error at
every build. (The problem is with the '\#\#' escape.)

Also drop UPSTREAM_VERSION (unused).

392c05d... by Paride Legovini on 2020-10-16

deb packaging: fully cleanup directory tree after build

LP: #1899698