Merge ~mwhudson/curtin:default-first-lba into curtin:master
Proposed by
Michael Hudson-Doyle
Status: | Merged |
---|---|
Approved by: | Dan Bungert |
Approved revision: | 02bc28d88d3556a292f9d0dad9368ba0f67b128b |
Merge reported by: | Server Team CI bot |
Merged at revision: | not available |
Proposed branch: | ~mwhudson/curtin:default-first-lba |
Merge into: | curtin:master |
Diff against target: |
53 lines (+29/-2) 2 files modified
curtin/commands/block_meta_v2.py (+8/-2) tests/integration/test_block_meta.py (+21/-0) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Dan Bungert | Approve | ||
Server Team CI bot | continuous-integration | Approve | |
Review via email: mp+433618@code.launchpad.net |
Commit message
block_meta_v2: set first-lba if needed when creating a new GPT
sfdisk sets first-lba to 2048 by default, which fails if one of
the partitions has an start smaller than that. Having a
partition start so early in the disk is a bit strange but it
works if you are preserving a partition (because we preserve
first-lba) so it should probably work when creating one too.
To post a comment you must log in.
PASSED: Continuous integration, rev:68d084399c4 6ad57aa25b1f44c d3f06444100f98 /jenkins. canonical. com/server- team/job/ curtin- ci/55/ /jenkins. canonical. com/server- team/job/ curtin- ci/nodes= metal-amd64/ 55/ /jenkins. canonical. com/server- team/job/ curtin- ci/nodes= metal-arm64/ 55/ /jenkins. canonical. com/server- team/job/ curtin- ci/nodes= metal-ppc64el/ 55/ /jenkins. canonical. com/server- team/job/ curtin- ci/nodes= metal-s390x/ 55/
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
Click here to trigger a rebuild: /jenkins. canonical. com/server- team/job/ curtin- ci/55// rebuild
https:/