Project

General

Profile

Actions

Bug #2792

open

If you delete "PXE Localboot Default" cancel build fails

Added by Florian Rosenegger almost 11 years ago. Updated over 10 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
Templates
Target version:
-
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

Hello,

i deleted the Provisioning template "PXE Localboot Default" and after that it was no longer possible to cancel a build for a host.

The error i got was:
Failed to save: Failed to generate PXELinux template: undefined method `template' for nil:NilClass

Via finish.sh script i also tried to call https://foreman/unattended/built but i only got a 409.

So maybe it would be good to make the Template undeleteable.

Thanks,
Florian


Related issues 4 (1 open3 closed)

Related to Foreman - Feature #3103: Read only provisioning/config templates for pluginsClosedStephen Benjamin09/17/2013Actions
Related to Foreman - Feature #7484: Per-subnet PXELinux global default filesNew09/17/2014Actions
Has duplicate Foreman - Bug #2215: When PXE Localboot Default is deleted, causes nil error when cancelling buildDuplicate02/17/2013Actions
Has duplicate Foreman - Bug #4008: Disable 'build mode' unsuccessfulDuplicate01/14/2014Actions
Actions #1

Updated by Greg Sutcliffe over 10 years ago

As we don't have an "undeleteable" feature for templates, I'm not sure this is a strong enough use case to implement one. It could be sensible to have some internal string which could be used if the template is not found, though. Any objections?

Actions #2

Updated by Lukas Zapletal over 10 years ago

  • Category set to Templates
  • Target version set to 1.3.0

Can't we just implement ON DELETE ERROR instead default ON DELETE NULL for this one?

Actions #3

Updated by Greg Sutcliffe over 10 years ago

We could, yes. I'm happy with either approach, tbh.

Actions #4

Updated by Florian Rosenegger over 10 years ago

Hello again,

i would also be happy if Foreman would just post a message that it is missing this special template (maybe with a hint to create one with the default name..)

My problem was that i just couldn't cancel the build but there was none whatsoever reason why (even in the log) and without Ohad and the IRC i wouldn't had a single idea what the error means.

The undeleteable feature for important templates was just one idea, but maybe/definitely not the best.

Actions #5

Updated by Lukas Zapletal over 10 years ago

  • Description updated (diff)
  • Target version deleted (1.3.0)
Actions #6

Updated by Dominic Cleal over 10 years ago

  • Has duplicate Bug #2215: When PXE Localboot Default is deleted, causes nil error when cancelling build added
Actions #7

Updated by Dominic Cleal over 10 years ago

  • Related to Feature #3103: Read only provisioning/config templates for plugins added
Actions #8

Updated by Lukas Zapletal over 10 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Lukas Zapletal
  • Target version set to 1.15.0
  • translation missing: en.field_release set to 1
  • Difficulty set to trivial

At least, I am adding comment to the templates.

https://github.com/theforeman/foreman/pull/977

Actions #9

Updated by Dominic Cleal over 10 years ago

  • Status changed from Ready For Testing to New
  • Assignee deleted (Lukas Zapletal)
  • Target version deleted (1.15.0)
  • translation missing: en.field_release deleted (1)
  • Difficulty deleted (trivial)

Sorry, I'm going to reset this as I don't think this is a final solution to the issue, helpful though it is. Thanks Lukas.

Actions #10

Updated by Dominic Cleal over 10 years ago

  • Has duplicate Bug #4008: Disable 'build mode' unsuccessful added
Actions #11

Updated by Dominic Cleal over 9 years ago

  • Related to Feature #7484: Per-subnet PXELinux global default files added
Actions

Also available in: Atom PDF