diff options
| author | Julia Kreger <juliaashleykreger@gmail.com> | 2020-05-14 17:43:08 -0700 |
|---|---|---|
| committer | Steve Baker <sbaker@redhat.com> | 2021-07-27 09:30:22 +1200 |
| commit | a5fc6cc2cba367ee941b1d810707e371083c7ebc (patch) | |
| tree | 36957dcce2143dcb1606289a7911b968065b1e93 /ironic/drivers/modules/ipxe.py | |
| parent | f1b87e82d0328a5750258879b680c930c944faf0 (diff) | |
| download | ironic-a5fc6cc2cba367ee941b1d810707e371083c7ebc.tar.gz | |
Provide a path to set explicit ipxe bootloaders
I did something stupid when started driving forth the split of ipxe
from the pxe interface: I didn't think about the need to actually
separate bootloaders. In part, because the use case was a mixed
Power8/Power9 and x86 cluster. Mainly because the Power hardware
does not honor or care about the bootfile name provided over DHCP.
The firmware knows how to read the PXELINUX boot file format
and the machines are able to boot from there.
Where this all goes sideways is when:
* Enabled boot interfaces are set to ipxe,pxe
* No default boot interface is set
* Node is created without a default for x86 hardware.
* Node uses ipxe boot_interface, and creates files under /httpboot
* bootfile transmitted via DHCP is pxelinux.0.
Fun right?
The simple workaround for the power user is to just define the iPXE
loader, or maybe use UEFI. But that is neither here nor there, this
is still a bug and a possible use case is GRUB2 via PXE and iPXE.
Not that would really work via ipxe, but hopefully people get the
idea.
The solution kind of seems clear, duplicate configuration and
fallback if not defined.
This is backported to stable/train as a prerequisite to backporting
https://review.opendev.org/c/openstack/puppet-ironic/+/797683
Story: #2007003
Task: #40282
Change-Id: I4419254c23095929e52a0fda11789f2f5167dc6b
(cherry picked from commit 5f7d84f483be165fde04954453e156ceefc43d28)
Diffstat (limited to 'ironic/drivers/modules/ipxe.py')
| -rw-r--r-- | ironic/drivers/modules/ipxe.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ironic/drivers/modules/ipxe.py b/ironic/drivers/modules/ipxe.py index bfe1d1cd4..bbf06e3be 100644 --- a/ironic/drivers/modules/ipxe.py +++ b/ironic/drivers/modules/ipxe.py @@ -164,7 +164,7 @@ class iPXEBoot(pxe_base.PXEBaseMixin, base.BootInterface): ipxe_enabled=True) pxe_options.update(ramdisk_params) - pxe_config_template = deploy_utils.get_pxe_config_template(node) + pxe_config_template = deploy_utils.get_ipxe_config_template(node) pxe_utils.create_pxe_config(task, pxe_options, pxe_config_template, |