-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Add compatibility file for GRUB versions up to v2.06 #15909
base: master
Are you sure you want to change the base?
Conversation
I am not sure this approach is really usable. Without knowing which GRUB versions are affected, it is impossible to safely choose the compatibility profile. And the lack of understanding for the issue makes its further maintenance problematic. Read-only-compatible features should not cause problems to GRUB, so I wonder if it is just a GRUB bug triggered by some of the features, or some feature is not so much compatible as we'd like. |
It was a bug they've fixed, yes, but I don't think we're going to win the argument of making every distro cherrypick grub2 fixes in a timely fashion, compared to people running our newest versions or people packaging newer things than the distro does, and getting burned. I said in another one, I think #15459, that I think we should do the opposite, and make Admittedly, this is biased by me not looking forward to having to explain to a lot of people who will come by asking why |
11ef525
to
2273c2b
Compare
I have updated the PR and added a new compatibility file, The existing grub2 compatibility file lists all read-only features that can be enabled on boot pool for grub with version 2.12 |
GRUB is not able to detect ZFS pool if snaphsot of top level boot pool is created. This issue is observed with GRUB versions up to v2.06 if extensible_dataset feature is enabled on ZFS boot pool. compatibility=grub2-2.06 would enable all read-only compatible zpool features except extensible_dataset and other features that depend on it. The existing grub2 compatibility file lists all read-only features that can be enabled on boot pool for grub with version 2.12 onwards. Signed-off-by: Umer Saleem <usaleem@ixsystems.com>
2273c2b
to
1040cfb
Compare
I have been working on testing all the zpool features present in While the naming convention of newly added file and whether we should rename this file to The issue is fixed in grub and working around it by dirty-ing the pool status for nearly all zpools using grub2 compatibility does not seem like a good solution to me. |
Motivation and Context
GRUB is not able to detect ZFS pool if snaphsot of top level boot
pool dataset is created. This issue is observed with GRUB versions
up to v2.06 if extensible_dataset feature is enabled on ZFS boot pool.
#15261
#13873
Description
A new compatibility file is added
grub2-2.06
for GRUB versionsupto 2.06.
compatibility=grub2-2.06
would enable all read-onlycompatible zpool features except
extensible_datase
t and otherfeatures that depend on it.
The existing grub2 compatibility file lists all read-only features
that can be enabled on boot pool for grub with version 2.12
onwards. The issue is fixed on GRUB v2.12.
Documented both
grub2
andgrub2-2.06
files in zpool-features.7.livelist
feature also depends onextensible_dataset
, but it wasdocumented. zpool-features.7 is updated for that as well.
How Has This Been Tested?
Tried creating boot pool with
compatibility=grub2-2.06
, aftercreating the snapshot of top level dataset of ZFS boot pool.
Verified
grub-probe
still detects the ZFS pool.Confirmed after re-enabling
extensible_dataset
feature and issueoccurs.
Types of changes
Checklist:
Signed-off-by
.