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

Fixed handling of oem reboot settings #2055

Merged
merged 1 commit into from
Feb 7, 2022
Merged

Fixed handling of oem reboot settings #2055

merged 1 commit into from
Feb 7, 2022

Conversation

schaefi
Copy link
Collaborator

@schaefi schaefi commented Feb 4, 2022

There are oem settings called oem-reboot, oem-reboot-interactive
as well as oem-shutdown and oem-shutdown-interactive. When used
the information is passed along to the profile but not evaluated
by any initrd code. I don't know where on the way we lost the
code that actually works with these settings but this commit
makes them effective.

@schaefi schaefi requested a review from davidcassany February 4, 2022 08:54
@schaefi schaefi self-assigned this Feb 4, 2022
@schaefi
Copy link
Collaborator Author

schaefi commented Feb 4, 2022

@davidcassany would be great if you find the time to look at this change. It's initrd code you know it's mean and always strikes back :) Thanks much.

I also set the WIP label as I wanted to test it extensively before merge

Copy link
Collaborator

@davidcassany davidcassany left a comment

Choose a reason for hiding this comment

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

Looks good! It is incredible we lost that at some point... 😓 Good this is all back. All these portions from XML to dracut modules are hard to track.

There are oem settings called oem-reboot, oem-reboot-interactive
as well as oem-shutdown and oem-shutdown-interactive. When used
the information is passed along to the profile but not evaluated
by any initrd code. I don't know where on the way we lost the
code that actually works with these settings but this commit
makes them effective. This Fixes #2056
@schaefi
Copy link
Collaborator Author

schaefi commented Feb 7, 2022

@davidcassany Thanks for looking at it. I have done some tests and it worked in the desired way. I will merge this as soon as the tests passes

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.

2 participants