Project

General

Profile

Bug #29239

Default PXE global template entry "discovery" no longer works for EFI systems

Added by Lukas Zapletal 3 months ago. Updated about 1 month ago.

Status:
Closed
Priority:
Normal
Category:
Templates
Target version:

Description

We have changed global template to this:

menuentry 'Foreman Discovery Image <%= suffix %>' --id discovery<%= suffix %> {
  linux<%= suffix %> boot/fdi-image/vmlinuz0 rootflags=loop root=live:/fdi.iso rootfstype=auto ro rd.live.image acpi=force rd.luks=0 rd.md=0 rd.dm=0 rd.lvm=0 rd.bootif=0 rd.neednet=0 nokaslr nomodeset proxy.url=<%= foreman_server_url %> proxy.type=foreman BOOTIF=01-$mac
  initrd<%= suffix %> boot/fdi-image/initrd0.img
}

By adding <%= suffix %> to the --id discovery, it is no longer possible to force systems to load the discovery image using the "Default PXE global template entry" in systems mixed with UEFI and non-UEFI machines. Adding the <%= suffix %> to the menu item test makes sense, changing the --id not so much. Is there a reason why the --id section was changed?

Associated revisions

Revision 3f215666 (diff)
Added by Lukas Zapletal 2 months ago

Fixes #29239 - unify discovery grub2 menu items

History

#1 Updated by Lukas Zapletal 3 months ago

  • Target version deleted (Discovery Plugin 16.0)
  • Category deleted (Discovery plugin)
  • Project changed from Discovery to Templates

#2 Updated by Lukas Zapletal 3 months ago

  • Category set to Templates
  • Project changed from Templates to Foreman
  • Fixed in Releases 2.1.0 added

#3 Updated by Lukas Zapletal 3 months ago

  • Target version set to 2.1.0
  • Fixed in Releases deleted (2.1.0)

#4 Updated by Lukas Zapletal 3 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/community-templates/pull/696 added

#5 Updated by The Foreman Bot 3 months ago

  • Assignee set to Lukas Zapletal

#6 Updated by Lukas Zapletal 2 months ago

  • Target version changed from 2.1.0 to 2.0.0

Actually this is a regression it seems, I will try to hit 2.0 for this one.

#7 Updated by The Foreman Bot 2 months ago

  • Fixed in Releases 2.1.0 added

#8 Updated by Anonymous 2 months ago

  • Status changed from Ready For Testing to Closed

#9 Updated by The Foreman Bot 2 months ago

  • Pull request https://github.com/theforeman/community-templates/pull/714 added

#10 Updated by Tomer Brisker about 2 months ago

  • Fixed in Releases 2.0.0 added
  • Fixed in Releases deleted (2.1.0)

#11 Updated by The Foreman Bot about 2 months ago

  • Pull request https://github.com/theforeman/community-templates/pull/717 added

Also available in: Atom PDF