Merge ~mwhudson/curtin:fix-DasdPartitionTable into curtin:master
Proposed by
Michael Hudson-Doyle
Status: | Merged |
---|---|
Approved by: | Michael Hudson-Doyle |
Approved revision: | 70954686adb559e8bdccbf68cc4a44e19ec604da |
Merge reported by: | Server Team CI bot |
Merged at revision: | not available |
Proposed branch: | ~mwhudson/curtin:fix-DasdPartitionTable |
Merge into: | curtin:master |
Diff against target: |
149 lines (+74/-28) 3 files modified
curtin/block/dasd.py (+32/-27) tests/unittests/test_block_dasd.py (+41/-0) tests/unittests/test_commands_block_meta.py (+1/-1) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Paride Legovini | Approve | ||
Server Team CI bot | continuous-integration | Approve | |
Review via email: mp+394985@code.launchpad.net |
Commit message
fix construction of DasdPartitionTable from fdasd output
DasdPartition had an extra argument in its constructor that should not
have been there. Add some tests.
To post a comment you must log in.
FAILED: Continuous integration, rev:e02d09d1000 b64532683a41448 a1b8781b58cb86 /jenkins. ubuntu. com/server/ job/curtin- ci/28/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-amd64/ 28/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-arm64/ 28/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-ppc64el/ 28/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-s390x/ 28/
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/28// rebuild
https:/