lp://staging/~wesley-wiedenmeier/curtin/fix-extended-clear-holders

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp://staging/~curtin-dev/curtin/trunk at revision 420

Server Team CI bot: Approve (continuous-integration) on 2016-08-21

Wesley Wiedenmeier (community): Needs Resubmitting on 2016-06-15

curtin developers: Pending requested 2016-05-14

Diff: 2214 lines (+1515/-265)

15 files modified

curtin/block/__init__.py (+176/-47)
curtin/block/clear_holders.py (+379/-0)
curtin/block/lvm.py (+96/-0)
curtin/commands/block_info.py (+75/-0)
curtin/commands/block_meta.py (+82/-203)
curtin/commands/block_wipe.py (+0/-1)
curtin/commands/clear_holders.py (+48/-0)
curtin/commands/main.py (+4/-3)
curtin/config.py (+2/-3)
curtin/util.py (+35/-8)
doc/devel/clear_holders_doc.txt (+85/-0)
tests/unittests/test_block.py (+99/-0)
tests/unittests/test_block_lvm.py (+94/-0)
tests/unittests/test_clear_holders.py (+321/-0)
tests/unittests/test_util.py (+19/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1562003: curtin has trouble removing partitions on redeployment	Medium	Fix Released
Bug #1579572: Curtin disk partitioning fails with storage config containing a msdos extended/logical partition scheme using the 'wipe' flag	Undecided	Fix Released
Bug #1588875: [2.0-b6] Deploying a trusty (but not xenial) node frequently fails during storage setup of curtin	Undecided	Fix Released
Bug #1591573: Curtin clear_holders() does not work on lvm devices with either their vg name or lv name containing a -	Undecided	Fix Released
Bug #1592962: block.wipe_volume(mode='pvremove') fails on ubuntu precise	Undecided	Fix Released
Bug #1611208: Curtin is unable to install to disks which previously contained a dm-crypt device	Undecided	Fix Released

Link a bug report

Related blueprints

523. By Wesley Wiedenmeier on 2016-08-21

Merge from trunk

522. By Wesley Wiedenmeier on 2016-08-19

Merge from lp:curtin

521. By Wesley Wiedenmeier on 2016-08-15

Add function lvm.lvmetad_running() to check status of lvm metadata daemon and
only append '--cache' flag in lvm.lvm_scan() if it is running. this avoids
issues when the lvmetad service is not running on a release that does support
it

520. By Wesley Wiedenmeier on 2016-08-11

In block.clear_holders, do not define DEV_TYPES registry at bottom of file.
Instead, put its contents into a function at the top that is run by the
interpereter when it reaches the bottom of the file. This makes the contents of
the registry easy to find and read for anyone viewing the file without causing
errors because the interpreter is attempting to find functions that have not
been defined yet as would happen if the registry were moved to the top of the
file outside ofa function

519. By Wesley Wiedenmeier on 2016-08-11

In util.write_file, don't do chmod if mode kwarg is a false value
In block.clear_holders.shutdown_bcache use util.write_file

518. By Wesley Wiedenmeier on 2016-08-11

Accept a failure to run 'modprobe bcache' instead of halting installation
because any systems that have a kernel that does not support bcache will never
start bcache devies even if there are bcache superblocks present on the system
already, so it is okay if clear_holders does not detect the old bcache
superblocks as they won't actually hold anything back from being partitioned

517. By Wesley Wiedenmeier on 2016-08-11

Updated clear_holders based on diff comments in
https://code.launchpad.net/~wesley-wiedenmeier/curtin/fix-extended-clear-holders/+merge/294694

516. By Wesley Wiedenmeier on 2016-08-11

In clear_holders.shutdown_bcache test if device to shut down is already shut
down before doing anything

515. By Wesley Wiedenmeier on 2016-08-10

IN clear_holders.wipe_superblock, use blockdev not sysfs path in call to
block.is_extended_partition

514. By Wesley Wiedenmeier on 2016-08-10

Add unittest for last update to check_efi_signature