Merge lp://staging/~blake-rouse/maas-images/fix-centos-efi into lp://staging/maas-images

Proposed by Blake Rouse
Status: Merged
Merged at revision: 361
Proposed branch: lp://staging/~blake-rouse/maas-images/fix-centos-efi
Merge into: lp://staging/maas-images
Diff against target: 68 lines (+43/-3)
1 file modified
curtin/centos7/curtin-hooks.py (+43/-3)
To merge this branch: bzr merge lp://staging/~blake-rouse/maas-images/fix-centos-efi
Reviewer Review Type Date Requested Status
Andres Rodriguez (community) Approve
Review via email: mp+323594@code.staging.launchpad.net

Commit message

Fix boot loader ordering for CentOS 7 UEFI.

Previously when deploying CentOS 7 the 'centos' efi loader would become the first loader in the boot order, which prevents re-deployments of any OS as the machine will no longer boot from the network first. This branch updates the boot order for EFI and places the currently booted entry as the first in the boot order. Since this is run in ephemeral mode this ensures that the NIC that the machine booted from will start first. 'centos' entry will be after the network, so if MAAS is down the machine will still boot from 'centos' if MAAS is down and the machine reboots. Previous \EFI entries are also removed from bios as those no longer exists as curtin creates a new \EFI partition on deployment.

To post a comment you must log in.
Revision history for this message
Andres Rodriguez (andreserl) wrote :

lgtm!

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