Merge ~mwhudson/curtin:always-devpath into curtin:master
Proposed by
Michael Hudson-Doyle
Status: | Needs review |
---|---|
Proposed branch: | ~mwhudson/curtin:always-devpath |
Merge into: | curtin:master |
Diff against target: |
70 lines (+12/-7) 3 files modified
curtin/commands/block_meta.py (+9/-7) tests/unittests/test_commands_block_meta.py (+1/-0) tests/unittests/test_curthooks.py (+2/-0) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Dan Bungert | Approve | ||
Server Team CI bot | continuous-integration | Approve | |
Review via email: mp+415876@code.staging.launchpad.net |
Commit message
block_meta: have get_path_
Specifically, return /dev/dm-X for device-mapper paths, not /dev/mapper/foo.
I thought this would fix the multipath reuse vmtests but it didn't.
It still seems like a good idea though.
Description of the change
I took a look at the failing vmtests and I think this is the only change needed for curtin (vs changes to test data or code).
Curtin really needs a block device object.
To post a comment you must log in.
There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.
FAILED: Continuous integration, rev:96b8b5b9d43 b2a5016951b26c6 cfe9f72d82af82 /jenkins. ubuntu. com/server/ job/curtin- ci/250/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-amd64/ 250/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-arm64/ 250/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-ppc64el/ 250/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-s390x/ 250/
https:/
Executed test runs:
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
Click here to trigger a rebuild: /jenkins. ubuntu. com/server/ job/curtin- ci/250/ /rebuild
https:/