Project

General

Profile

Actions

Bug #14521

closed

TFTP orchestration fails for OS with iPXE w/o PXELinux associated

Added by Lukas Zapletal over 8 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Category:
Orchestration
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

For a host with OS with iPXE template but no PXELinux template, we still attempt to render PXELinux template which leads to "Failed to generate PXELinux template: undefined method `encoding' for nil:NilClass" orchestration error.

We should give nicer error instead.


Related issues 2 (0 open2 closed)

Related to Foreman - Bug #612: Host#configTemplate does not work as expectedClosedOhad Levy01/25/2011Actions
Related to Foreman - Feature #1294: Build should not require TFTP if using gPXEClosedOhad Levy11/03/2011Actions
Actions #1

Updated by Lukas Zapletal over 8 years ago

We already have a check but that does not trigger when there is an iPXE template associated, that is the case:

    if host.provisioning_template({:kind => host.operatingsystem.template_kind}).nil? && host.provisioning_template({:kind => "iPXE"}).nil?
      failure _("No %{template_kind} templates were found for this host, make sure you define at least one in your %{os} settings") %
                { :template_kind => host.operatingsystem.template_kind, :os => host.operatingsystem }
    end

Actions #2

Updated by Lukas Zapletal over 8 years ago

  • Related to Bug #612: Host#configTemplate does not work as expected added
Actions #3

Updated by Lukas Zapletal over 8 years ago

  • Related to Feature #1294: Build should not require TFTP if using gPXE added
Actions #4

Updated by Lukas Zapletal over 8 years ago

  • Subject changed from Queue TFTP orchestration only when there is PXELinux associated to TFTP orchestration fails for OS with iPXE w/o PXELinux associated
Actions #5

Updated by Lukas Zapletal over 8 years ago

  • Description updated (diff)
Actions #6

Updated by The Foreman Bot over 8 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/3402 added
Actions #7

Updated by Lukas Zapletal over 8 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions #8

Updated by Dominic Cleal over 8 years ago

  • Translation missing: en.field_release set to 136
Actions

Also available in: Atom PDF