Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support legacy MBR (msdos) as part of UEFI to enable hybrid builds #763

Closed
wants to merge 62 commits into from

Conversation

nullr0ute
Copy link
Contributor

The UEFI spec supports legacy MBR (msdos) in section 5.2 of the UEFI 2.7 spec [1].
There's a number of use cases where we might boot on a UEFI device but want to
support, but not by default, the creation of images using legacy MBR partitions.
One of these use cases is cloud images where we can produce a single image
tha will run on UEFI supported, including secure-boot, clouds but also
support AWS which only supports MBR partitioning schemes [2] and in the current
config anaconda fails the install where this is a perfectly valid and widely
supported deployment mechanism.

So enable the ability to use legacy msdos partitioning schemes, but don't use
it for default installs, so people may consume this mechanism using kickstarts
rather than having the install fail for a valid use case.

[1] https://uefi.org/sites/default/files/resources/UEFI%20Spec%202_7_A%20Sept%206.pdf
[2] https://docs.aws.amazon.com/vm-import/latest/userguide/vmie_prereqs.html

Signed-off-by: Peter Robinson pbrobinson@gmail.com

bcl and others added 30 commits March 11, 2016 17:55
The tests run the translation-canary test which needs the translations,
so this makes sure you can pull them down for the test.
We don't want to use the information from the system the tests are running on
and actually we cannot even grab that information because the tests may not be
run as root. To prevent issues and exceptions, we can easily just mock all three
LVM listing functions - lvs(), vgs() and pvs().
…_vgs

Mock all blockdev's listing functions for populator tests
…nk_size

Allow custom chunk size specification for MDRaidArrayDevice
Also remove l10n lines.

(cherry picked from commit b9ec796)
dwlehman and others added 22 commits November 8, 2017 22:47
(This reverts commit 495be07,
which removed the nvdimm Recommends on 3.0-devel since nvdimm
support does not exist on that branch.)
Spec file updates for python 2/3 and Fedora dist-git.
The UEFI spec supports legacy MBR (msdos) in section 5.2 of the UEFI 2.7 spec [1].
There's a number of use cases where we might boot on a UEFI device but want to
support, but not by default, the creation of images using legacy MBR partitions.
One of these use cases is cloud images where we can produce a single image
tha will run on UEFI supported, including secure-boot, clouds but also
support AWS which only supports MBR partitioning schemes [2] and in the current
config anaconda fails the install where this is a perfectly valid and widely
supported deployment mechanism.

So enable the ability to use legacy msdos partitioning schemes, but don't use
it for default installs, so people may consume this mechanism using kickstarts
rather than having the install fail for a valid use case.

[1] https://uefi.org/sites/default/files/resources/UEFI%20Spec%202_7_A%20Sept%206.pdf
[2] https://docs.aws.amazon.com/vm-import/latest/userguide/vmie_prereqs.html

Signed-off-by: Peter Robinson <pbrobinson@gmail.com>
@vojtechtrefny vojtechtrefny changed the base branch from 3.1-release to 3.1-devel March 8, 2019 10:18
@vojtechtrefny vojtechtrefny changed the base branch from 3.1-devel to 3.1-release March 8, 2019 10:18
@vojtechtrefny
Copy link
Member

Can you please create a new PR against 3.1-devel, we have compilcated system of branches and the release ones are only for doing releases. I've tried to change the branch using GH, but it doesn't wort.

@nullr0ute nullr0ute changed the base branch from 3.1-release to 3.1-devel March 8, 2019 10:53
@nullr0ute
Copy link
Contributor Author

It would be useful if you'd actually document that somewhere, it would save everyone a bunch of time :)

@nullr0ute nullr0ute closed this Mar 8, 2019
@nullr0ute
Copy link
Contributor Author

#764

@vojtechtrefny
Copy link
Member

We have some documentation for contributors but it's a little bit outdated. Thank you for creating the new PR:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants