Feature #19554
closed
Provide warning for unsupported PXE loader combinations
Description
Users very often try to provision RHEL 6 with Grub2 in UEFI which leads to boot loading errors (bacically Grub2 prompt or kernel panic).
We need a warning next to PXE Loader flag in the UI if there is known combination of OS and loader that does not work. These currently are:
- RHEL 6.x and Grub2
- RHEL 7.x and Grub1
- Ubuntu 10.x or older and Grub2
- Ubuntu 11.x or newer and Grub1
- Debian 2-6 and Grub2
- Debian 7+ and Grub1
This can be as simple as regular expression and a warning message showing up next to the field in the UI. No actions must be taken. A global setting should be present which will turn this feature of in case user do not want to see these warnings.
Updated by Lukas Zapletal over 7 years ago
Good idea would be our status API to do this kind of warnings.
Updated by Tomáš Strachota over 7 years ago
- Status changed from New to Assigned
- Assignee set to Tomáš Strachota
Updated by Lukas Zapletal over 7 years ago
Just to clear things little bit, these are known and supported cobinations, we should only issue warning if ANY of these rules are not true:
- RHEL 6.x and Grub1
- RHEL 7.x and Grub2
- Ubuntu 10.x or older and Grub1
- Ubuntu 11.x or newer and Grub2
- Debian 2-6 and Grub1
- Debian 7+ and Grub2
It's actually the opposite of unsupported combinations in the original description. I think we should have a list of supported pairs rather than unsupported, that's all.
Updated by Marek Hulán over 7 years ago
- Target version changed from 1.17.0-RC2 to 1.18.0-RC2
Updated by The Foreman Bot over 7 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/4792 added
Updated by Marek Hulán over 7 years ago
- Target version changed from 1.18.0-RC2 to 214
Updated by Anonymous over 7 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset 805420d18ae67be2fa76cc433ef58b685a640afa.
Updated by Daniel Lobato Garcia over 7 years ago
- Translation missing: en.field_release set to 287
Given the entirety of the check is contained in a class "PXECompatibilityCheck" added in this PR, I'm marking as 1.15.4 as it's very useful.
Updated by The Foreman Bot over 7 years ago
- Pull request https://github.com/theforeman/foreman/pull/4810 added
Updated by The Foreman Bot over 7 years ago
- Pull request https://github.com/theforeman/foreman/pull/4811 added
Updated by Daniel Lobato Garcia over 7 years ago
- Translation missing: en.field_release changed from 287 to 296
Updated by Ohad Levy over 7 years ago
- Related to Refactor #20846: refactor usage of ES6 Map to using _set added
Updated by Ohad Levy over 7 years ago
- Pull request deleted (
https://github.com/theforeman/foreman/pull/4810)
Updated by