lp:~smoser/curtin/trunk.lp1684128

Branch merges

Propose for merging

No branches dependent on this one.

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

Ryan Harper (community): Approve on 2017-04-20

Server Team CI bot: Approve (continuous-integration) on 2017-04-20

Joshua Powers (community): Approve on 2017-04-20

Diff: 67 lines (+17/-10)

2 files modified

curtin/commands/install.py (+13/-9)
tests/vmtests/test_basic.py (+4/-1)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1684128: curtin install log not copied to target

Medium

Fix Released

Link a bug report

Related blueprints

492. By Scott Moser on 2017-04-20

copy content in binary mode only.

if the install log had content that was not decodable, then this
would fail.

491. By Scott Moser on 2017-04-19

move copy later

490. By Scott Moser on 2017-04-19

fix pep8

489. By Scott Moser on 2017-04-19

fix error

488. By Scott Moser on 2017-04-19

Copy curtin install log into target.

The install log was not correctly being copied into the target.
Now, by default the target should get:
  install log: /root/curtin-install.log
  merged config: /root/curtin-install-cfg.yaml

487. By Joshua Powers on 2017-04-13

vmtest: update source.list test

The change that was made to the builtin /etc/apt/sources.list in the
cloud images for xenial was done so recently under bug LP: #1513529.
That was by design and required for consistency and this change updates
the curtin vmtest accordingly.

LP: #1682516

486. By Scott Moser on 2017-04-12

add recursive flag to do_umount, add --umount to block-meta

This adds a 'recursive' argument to util.do_umount, and utilizes
it from install.py. The basic idea of do_umount with recursive
is to walk through /proc/mounts in reverse and unmount anything
mounted under the provided mountpoint (or the mountpoint itself).

Also added here is use of that function from block-meta command.
We add a flag --umount to unmount filesystems at its end.

485. By Scott Moser on 2017-04-07

load_command_environment: fix strict flag, and unit tests.

The strict=True option to load_command_environment was broken.
Instead of checking for the expected environment variables in the
environment, it would check the state names.

484. By Ryan Harper on 2017-04-07

block_meta: Attempt to mount by UUID= if possible

When generating fstab entries in mount_handler prefer UUID if the
underlying volume has an identifiable UUID available.

Update vmtests to check by UUID in test_basic.

Fixes LP:#1676991

483. By Scott Moser on 2017-04-06

Make systems with root on bcache use root=UUID=.

Systems with root on bcache would end up booting with root=/dev/bcache0
(or bcache1). Bcache device names are not guaranteed, so this could
cause issues.

The fix is to invoke 'udevadm trigger' after creating a filesystem on
a bcache device. That will populate /dev/disk/by-uuid, which will
cause grub to properly decide to boot by root=UUID=.