curtin

Merge lp://qastaging/~wesley-wiedenmeier/curtin/1597522 into lp://qastaging/~curtin-dev/curtin/trunk

  1. 1597522
  2. Merge into trunk
Proposed by Wesley Wiedenmeier
Status: Merged
Merged at revision: 407
Proposed branch: lp://qastaging/~wesley-wiedenmeier/curtin/1597522
Merge into: lp://qastaging/~curtin-dev/curtin/trunk
Diff against target: 82 lines (+25/-6)
3 files modified
curtin/block/mkfs.py (+9/-5)
examples/tests/uefi_basic.yaml (+15/-0)
tests/vmtests/test_uefi_basic.py (+1/-1)
To merge this branch: bzr merge lp://qastaging/~wesley-wiedenmeier/curtin/1597522
Reviewer Review Type Date Requested Status
Ryan Harper (community) Approve
Server Team CI bot continuous-integration Approve
Review via email: mp+298719@code.qastaging.launchpad.net

Description of the change

Only append '-s 1' flag to mkfs commands when creating vfat volume on advanced format disk.
Also, support specifying sector size to mkfs.xfs and properly format argument.

Fixes (LP: 1597522)

To post a comment you must log in.
Revision history for this message
Server Team CI bot (server-team-bot) wrote :

PASSED: Continuous integration, rev:401
https://server-team-jenkins.canonical.com/job/curtin-ci/288/
Executed test runs:
    None: https://server-team-jenkins.canonical.com/job/generic-update-mp/285/console

Click here to trigger a rebuild:
https://server-team-jenkins.canonical.com/job/curtin-ci/288/rebuild

review: Approve (continuous-integration)
lp://qastaging/~wesley-wiedenmeier/curtin/1597522 updated
402. By Wesley Wiedenmeier

Add partition 3 on main disk to disks_to_check in test_uefi_basic

Revision history for this message
Server Team CI bot (server-team-bot) wrote :

PASSED: Continuous integration, rev:402
https://server-team-jenkins.canonical.com/job/curtin-ci/289/
Executed test runs:
    None: https://server-team-jenkins.canonical.com/job/generic-update-mp/286/console

Click here to trigger a rebuild:
https://server-team-jenkins.canonical.com/job/curtin-ci/289/rebuild

review: Approve (continuous-integration)
Revision history for this message
Ryan Harper (raharper) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches