openstack/ironic
Code Issues Proposed changes

Fix grub config path default

Browse Source 
Grub2 looks for files in different paths depending on the boot mode
of the binary. Previously the grub_config_path setting was defaulted
to the path used exclusively for BIOS booting, which meant anyone
using it had to override the setting. Now, we've set the default
to the default for UEFI booting, and the world should be a happier,
and less override filled place.

Change-Id: Id6723e92efb62f8ca03099f15c90580cec887ddd
This commit is contained in:
Julia Kreger 2023-01-03 16:40:16 -08:00
parent 1d07be8237
commit 2f4a156d29
2 changed files with 15 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ironic/conf/default.py
View File

@ -216,7 +216,7 @@ image_opts = [
'common/isolinux_config.template'),
help=_('Template file for isolinux configuration file.')),
cfg.StrOpt('grub_config_path',
default='/boot/grub/grub.cfg',
default='EFI/BOOT/grub.cfg',
help=_('GRUB2 configuration file location on the UEFI ISO '
'images produced by ironic. The default value is '
'usually incorrect and should not be relied on. '

14
releasenotes/notes/fix-grub2-uefi-config-path-f1b4c5083cc97ee5.yaml Normal file
View File

@ -0,0 +1,14 @@
---
fixes:
- |
Fixes the default value for the ``[DEFAULT]grub_config_path`` variable to
be the default path for UEFI bootloader configurations, where as the
default was previously the BIOS grub2 configuration path.
upgrades:
- |
The default configuration value for ``[DEFAULT]grub_config_path`` has
been changed from ``/boot/grub/grub.conf`` to ``EFI/BOOT/grub.efi`` as
the configuration parameter was for UEFI boot configuration, and the
``/boot/grub/grub2.conf`` path is for BIOS booting. This was verified
by referencing several working UEFI virtual media examples where this
value was overridden to the new configuration value.