openstack/ironic
Code Issues Proposed changes

Fix iPXE template for whole disk image

Browse Source 
The goto :boot_whole_disk section in the iPXE template is bogus, the
"kernel" command is being used to chain the "chain.c32" file, that
command should be "chain" and the "append" command is not a valid iPXE
command.

This patch replaces those commands with the "sanboot" command from iPXE
which is (among other things) used to chain the boot process to the
local disk.

Closes-Bug: #1524403
Change-Id: I7d2310d94e40b1c9c76957382ab19c0076bf42cd
This commit is contained in:
Lucas Alvares Gomes 2015-12-09 16:17:30 +00:00
parent 44ad4acc2c
commit 952d5c5a06
3 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
ironic/drivers/modules/ipxe_config.template
View File

@ -16,6 +16,4 @@ initrd {{ pxe_options.ari_path }}
boot
:boot_whole_disk
kernel chain.c32
append mbr:{{ DISK_IDENTIFIER }}
boot
sanboot --no-describe

4
ironic/tests/unit/drivers/ipxe_config.template
View File

@ -16,6 +16,4 @@ initrd http://1.2.3.4:1234/ramdisk
boot
:boot_whole_disk
kernel chain.c32
append mbr:{{ DISK_IDENTIFIER }}
boot
sanboot --no-describe

4
releasenotes/notes/fix-ipxe-template-for-whole-disk-image-943da0311ca7aeb5.yaml Normal file
View File

@ -0,0 +1,4 @@
---
fixes:
- Fixes an issue with the provided iPXE template where whole disk images
could not be booted. See https://bugs.launchpad.net/ironic/+bug/1524403.