lp:~raharper/curtin/trunk.fix-lp1718699

Created by Ryan Harper and last modified
Get this branch:
bzr branch lp:~raharper/curtin/trunk.fix-lp1718699
Only Ryan Harper can upload to this branch. If you are Ryan Harper please log in for upload directions.

Branch merges

Related bugs

Related blueprints

Branch information

Owner:
Ryan Harper
Project:
curtin
Status:
Merged

Recent revisions

541. By Ryan Harper

Reword vmtest test input

540. By Ryan Harper

Fix style

539. By Ryan Harper

merge from trunk

538. By Ryan Harper

Relocate vmtests for bug to new file

537. By Ryan Harper

Fix commands: block_wipe and clear_holders

536. By Ryan Harper

block: handle wiping bcache parts

In some cases block devices may have bcache parts which are owned by the
block layer but are not assembled into a bcache device. The result is that
curtin fails to get exclusive opens when attempting to wipe the device.
Resolve this by testing of a volume has a 'bcache' directory under sysfs
and issuing the appropriate stop commands.

It's also possible for "buried" bcache devices where have metadata at
particular partition offsets. Once curtin starts partitioning the device
the bcache kernel layer "finds" the parts and claims device ownership and
prevents wiping of the partition. We resolve this issue by having curtin
wipe 1M at the location of the partition before we actually create it.

LP: #1718699

535. By Ryan Harper

bcache: accept sysfs write failure in shutdown handler if path missing

Bcache device shutdown is asynchronus. The kernel may remove the
device after we've tested to see if the device needs to be stopped but
before we issue a 'stop' command to the bcache sysfs file.

When that happens, curtin throws an exception; this patch catches
those and tests if the target path is missing; and if so, ignores the
error as it's expected due to device removal.

All other write failures where the path is still present are still
valid and reported.

LP: #1700564

534. By Scott Moser

vmtest: Rephrase a message about no disks to be less scary

In the case where a test has passed and there are no disks to collect we
logged an 'error' however in that case it wasn't really an error.
This branch changes that message to a debug level bit of information.

533. By Ryan Harper

block_meta: use block.wipe_volume(mode=superblock) to clear MBR/GPT tables

In the case curtin is not directed to wipe the partition table via the
wipe: configuration, a disk may contain an MBR and sgdisk --clear does
not successfully wipe MBR tables. Replace the call to sgdisk with curtin
block.wipe_volume(mode=superblock) to clear both MBR and GPT

LP: 1722322

532. By Ryan Harper

vmtest: fix artful networking

- Remove skip_by_date calls in vmtests that are no longer valid
- ifenslave brings in ifupdown, so filter out ifenslave package if target
  release is artful (LP: #1706744)
- vlan and bridge-utils are not needed in artful, filter those packages from
  being installed in artful as well.
- Adjust artful bonding test to skip checking for ifslave and instead check
  that ifenslave is *not* installed
- Drop mtu settings in network-static-routes configuration, the test case
  only needs to test static routes
- Move static routes under an interface subnet to be compatible with
  netplan format which requires routes under an interface
- Refactor ip_route_show parsing, ignoring default route and fetching
  variable settings like 'proto'.
- Skiptest bridging in artful, need a cloud-init fix for stp in netplan
  (LP: #1721157)
- Skiptest for mtu in artful, need networkd to support mtu6 support
  (LP: #1671951)

Fixes: LP:#1714028 LP:#1718216, LP:#1706744

Branch metadata

Branch format:
Branch format 7
Repository format:
Bazaar repository format 2a (needs bzr 1.16 or later)
Stacked on:
lp:~curtin-dev/curtin/trunk
This branch contains Public information 
Everyone can see this information.

Subscribers