Merge ~raharper/curtin:fix/mpath-partitions-missed-wiping-devices into curtin:master
Proposed by
Ryan Harper
Status: | Merged |
---|---|
Approved by: | Dan Watkins |
Approved revision: | 410d41920d7c1fdbe08f165bba2b93ddbce1cad6 |
Merge reported by: | Server Team CI bot |
Merged at revision: | not available |
Proposed branch: | ~raharper/curtin:fix/mpath-partitions-missed-wiping-devices |
Merge into: | curtin:master |
Diff against target: |
115 lines (+90/-3) 2 files modified
curtin/block/clear_holders.py (+4/-3) tests/unittests/test_clear_holders.py (+86/-0) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Dan Watkins (community) | Approve | ||
Server Team CI bot | continuous-integration | Approve | |
Review via email: mp+378340@code.launchpad.net |
Commit message
clear-holders: ensure we wipe device even if multipath enabled not not mp
On Ubuntu releases where multipath is enabled but not in-use some devices
would fail to be wiped due to logic bug. This was found in the
BcacheBasic test. Refactor this logic to ensure that we wipe either
the in-use multipath dm device or the block device value originally
supplied to the wipe function.
To post a comment you must log in.
PASSED: Continuous integration, rev:507498e2bfd 9c506f27b5b957b 0664d43beed62c /jenkins. ubuntu. com/server/ job/curtin- ci/3821/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-arm64/ 3821/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-ppc64el/ 3821/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-s390x/ 3821/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= torkoal/ 3821/
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
Click here to trigger a rebuild: /jenkins. ubuntu. com/server/ job/curtin- ci/3821/ /rebuild
https:/