~mwhudson/curtin:partition-verify-dasd

Branch merges

Propose for merging

Merged into curtin:master

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

Dimitri John Ledkov (community): Approve on 2020-11-18

curtin developers: Pending requested 2020-11-15

Diff: 402 lines (+160/-117)

4 files modified

curtin/block/dasd.py (+90/-103)
curtin/commands/block_meta.py (+22/-4)
tests/unittests/test_block_dasd.py (+0/-5)
tests/unittests/test_commands_block_meta.py (+48/-5)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

0dafa72... by Michael Hudson-Doyle on 2020-11-18

remove test for just-deleted code

640bb68... by Michael Hudson-Doyle on 2020-11-18

remove some dead code

5f649c4... by Michael Hudson-Doyle on 2020-11-17

work around a bug in old versions of mock

ec5836c... by Michael Hudson-Doyle on 2020-11-15

add partition_verify_fdasd and call it to verify a dasd partition

b5dbff6... by Michael Hudson-Doyle on 2020-11-13

rename partition_verify to partition_verify_sfdisk

partition_verify_vtoc is coming

a786627... by Michael Hudson-Doyle on 2020-11-13

block.dasd: factor out fdasd --table parsing to separate class

In the process, this changes adding a partition to a dasd to not rely on
dasdview, which is good because that does not work on a dasd passed via
virtio to a KVM.

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