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

grub.cfg: drop set pager=1 #3649

Merged
merged 1 commit into from
Oct 20, 2023
Merged

grub.cfg: drop set pager=1 #3649

merged 1 commit into from
Oct 20, 2023

Conversation

jlebon
Copy link
Member

@jlebon jlebon commented Oct 20, 2023

This was in the very first commit that added a GRUB config to this repo. If set, it causes GRUB to pause output if the screen is full of messages until the user presses a key.

That's just incompatible with automation, so nuke it. Any warnings from GRUB should end up in the serial console logs still.

One way this can happen is if booting from multipath. GRUB tries each path in turn until it can read from it. For each tried path that's non- optimized, it'll log a message. So if the device has a large enough number of paths, we can trigger the pager functionality and hang boot.

Probably fixes: https://issues.redhat.com/browse/OCPBUGS-20123

This was in the very first commit that added a GRUB config to this repo.
If set, it causes GRUB to pause output if the screen is full of messages
until the user presses a key.

That's just incompatible with automation, so nuke it. Any warnings from
GRUB should end up in the serial console logs still.

One way this can happen is if booting from multipath. GRUB tries each
path in turn until it can read from it. For each tried path that's non-
optimized, it'll log a message. So if the device has a large enough
number of paths, we can trigger the pager functionality and hang boot.

Probably fixes: https://issues.redhat.com/browse/OCPBUGS-20123
@jlebon
Copy link
Member Author

jlebon commented Oct 20, 2023

(And since this file is moving to bootupd: coreos/bootupd#548).

Copy link
Member

@dustymabe dustymabe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jlebon jlebon enabled auto-merge (rebase) October 20, 2023 15:28
@jlebon jlebon merged commit 0eb25d1 into coreos:main Oct 20, 2023
5 checks passed
@jlebon jlebon deleted the pr/grub-paging branch October 20, 2023 16:06
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

2 participants