Merge ~raharper/curtin:fix/lvm-over-multipath-partition-wipe into curtin:master
Proposed by
Ryan Harper
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Ryan Harper | ||||
Approved revision: | 2eca4183ebe4e1bdfecec20e3adc966bf2b08e16 | ||||
Merge reported by: | Server Team CI bot | ||||
Merged at revision: | not available | ||||
Proposed branch: | ~raharper/curtin:fix/lvm-over-multipath-partition-wipe | ||||
Merge into: | curtin:master | ||||
Diff against target: |
284 lines (+156/-7) 7 files modified
curtin/block/clear_holders.py (+4/-0) curtin/block/multipath.py (+2/-0) curtin/commands/block_meta.py (+4/-0) examples/tests/multipath-lvm-part-wipe.yaml (+125/-0) tests/unittests/test_clear_holders.py (+5/-5) tests/unittests/test_curthooks.py (+7/-2) tests/vmtests/test_multipath_lvm.py (+9/-0) |
||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Server Team CI bot | continuous-integration | Approve | |
Dan Watkins (community) | Approve | ||
Review via email: mp+382605@code.launchpad.net |
Commit message
lvm-over-multipath: handle lookups of multipath members
Subiquity provides a 'path' value for disks in storage config.
This triggered an edge case where curtin attempted to wipe the
underlying scsi disk *while* multipath is enabled resulting in
an error when attempting to get exclusive access to the disk.
This branch resolves this by checking if a disk or partition
is related to multipath and if so returning the device mapper
disk path.
LP: #1869075
To post a comment you must log in.
FAILED: Continuous integration, rev:97a3132b55b 1e8cdd5b1318c9b fa6bb3005b4dd3 /jenkins. ubuntu. com/server/ job/curtin- ci/43/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-amd64/ 43/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-arm64/ 43/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-ppc64el/ 43/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-s390x/ 43/
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
FAILURE: https:/
SUCCESS: https:/
Click here to trigger a rebuild: /jenkins. ubuntu. com/server/ job/curtin- ci/43// rebuild
https:/