~raharper/curtin:fix/block-meta-first-partition

Branch merges

Propose for merging

On hold for merging into curtin:master

Server Team CI bot: Approve (continuous-integration) on 2018-06-15

curtin developers: Pending requested 2018-06-15

Diff: 1506 lines (+1119/-156)

11 files modified

curtin/block/__init__.py (+1/-7)
curtin/commands/block_meta.py (+342/-145)
curtin/config.py (+12/-0)
examples/tests/mirrorboot-msdos-partition.yaml (+1/-0)
examples/tests/mirrorboot-uefi.yaml (+2/-1)
examples/tests/mirrorboot.yaml (+7/-1)
examples/tests/partition-order.yaml (+213/-0)
tests/unittests/test_commands_block_meta.py (+420/-0)
tests/unittests/test_make_dname.py (+4/-2)
tests/vmtests/__init__.py (+9/-0)
tests/vmtests/test_partition_order.py (+108/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

fd4686b... by Ryan Harper on 2018-06-15

Drop unused determine_partition_number and reduce vmtests for partition order

1dbaa96... by Ryan Harper on 2018-06-15

Clean up use of self.device, add mirrorboot unittest

5a9f61d... by Ryan Harper on 2018-06-15

Drop multi boot disk unittest, that's a valid use case

e04dae3... by Ryan Harper on 2018-06-15

Fix syntax error from merge

2fd6771... by Ryan Harper on 2018-06-15

Merge remote-tracking branch 'diglett/fix/block-meta-first-partition' into fix/block-meta-first-partition

24f33b2... by Ryan Harper on 2018-06-15

additional update partition unittests

ca5e941... by Ryan Harper on 2018-06-15

Fix Mirrorboot configurations with partition info update

Drop the multi-disk boot check, Mirrorboot is a supported use-case
where two disks are mirrored and so both disks need to have grub
installed in case one drive fails.

For RAID0 devices, one can create partitions on the md device, however
most of block_meta code expects that the 'device' under a partition
already exists. This is true for most partitions but since the md
device won't yet be created, we need to handle determining the logical
sector size for proper partition sector start, end and length values.
For raid0, we examine the raid config for devices which will compose
the raid and extract the logical sector size from the elements.

Finally, update the vmtest storage yaml to ensure they properly create
the same partitions needed for booting on both drives in the mirror.

0e71f0a... by Ryan Harper on 2018-06-14

Fix use of logical_sector_size info from disk device config

d443737... by Ryan Harper on 2018-06-13

Fix copying original configs not modified when updating partition info

87f7427... by Ryan Harper on 2018-06-13

vmtest: cover partition order test

Add a vmtest for partition order and the following changes

- block-meta:
  - write out the updated storage_config yaml and the
    OrderedDict id mapping as yaml
  - Add method to convert OrderedDict mapping to storage config
- vmtests:
  - move collection of /dev/disk/{by-id, by-dname, by-uuid},
    /proc/mounts, /proc/partitions, /etc/fstab to base class