lp://qastaging/~wesley-wiedenmeier/curtin/1618429

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp://qastaging/~curtin-dev/curtin/trunk at revision 424

Server Team CI bot: Approve (continuous-integration) on 2016-09-14

Ryan Harper (community): Approve on 2016-09-14

Diff: 177 lines (+64/-30)

4 files modified

curtin/block/clear_holders.py (+10/-2)
curtin/block/mdadm.py (+18/-5)
tests/unittests/test_block_mdadm.py (+28/-23)
tests/unittests/test_clear_holders.py (+8/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1618429: Curtin doesn't clean up previous MD configuration

High

Fix Released

Link a bug report

Related blueprints

431. By Wesley Wiedenmeier on 2016-09-14

Add unittest to ensure that errors are still raised for mdadm.mdadm_assemble if
running with ignore_errors=False

430. By Wesley Wiedenmeier on 2016-09-14

Add unit test for clear_holders.start_clear_holders_deps()

429. By Wesley Wiedenmeier on 2016-08-30

put warning in log if had to ignore errors in mdadm_assemble

428. By Wesley Wiedenmeier on 2016-08-30

Accept return code 1 and 2 from mdadm --assemble in mdadm.mdadm_assemble even
if running with ignore_error=False as they are usually not problems

427. By Wesley Wiedenmeier on 2016-08-30

Revert addition of block.wipe_volume mdadm mode as it should not be needed if
all mdadm metadata exists on the first and last 1024k of a deviec

426. By Wesley Wiedenmeier on 2016-08-30

Use 'ignore_error=True' for call to mdadm_assemble in
clear_holders.start_clear_holders_deps

425. By Wesley Wiedenmeier on 2016-08-30

In mdadm.mdadm_assemble, add parameter ignore_error to ignore all errors from
running assemble command. If command has non-zero exit and ignore_error is not
set, then util.ProcessExecutionError will be raised

424. By Wesley Wiedenmeier on 2016-08-30

Add wipe mode 'mdadm' using 'mdadm --zero-superblock' to block.wipe_volume

423. By Ryan Harper on 2016-08-29

curtin/doc: overhaul curtin documentation for readthedocs.org

Add additional documentation on custom storage and network configuration
Integrate some development documentation in Sphinx format and update
the build scripts for docs to import curtin version from python code.

422. By Ryan Harper on 2016-08-29

curtin.util: re-add support for RunInChroot

In merge committed to trunk in revno 404, we dropped util.RunInChroot.
That method was being used from curthooks in lp:maas-images project to
run commands in target via util.ChrootableTarget. In order to support
installation of old images that depend on RunInChroot we need to add
the method back.

This merge does that by wrapping ChrootableTarget class. It also adds
a couple unit tests to ensure that this keeps working.