Project

General

Profile

Actions

Bug #9695

open

Clearer template errors should be given during build review

Added by Martin Hovmöller about 9 years ago. Updated about 7 years ago.

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

Description

When putting a machine in build mode, if there is an error in the template the error message is very vague, like "Failure parsing <template name>: undefined method `<' for nil:NilClass.".
This makes debugging very hard. The error message should at least contain the line number for the line causing the error.


Related issues 2 (0 open2 closed)

Related to Foreman - Feature #746: Generate all the Host template when click on Build to avoid errors during installationClosedShlomi Zadok03/15/2011Actions
Related to Foreman - Bug #18345: Better rescue from syntax error in template renderingClosedMarek Hulán02/01/2017Actions
Actions #1

Updated by Dominic Cleal about 9 years ago

  • Project changed from Docker to Foreman
  • Subject changed from Clearer template errors should be given. to Clearer template errors should be given during build review
  • Category set to Templates
Actions #2

Updated by Dominic Cleal about 9 years ago

  • Related to Feature #746: Generate all the Host template when click on Build to avoid errors during installation added
Actions #3

Updated by Shlomi Zadok about 9 years ago

  • Assignee set to Shlomi Zadok
Actions #4

Updated by Shlomi Zadok over 7 years ago

  • Assignee deleted (Shlomi Zadok)
Actions #5

Updated by Marek Hulán about 7 years ago

  • Related to Bug #18345: Better rescue from syntax error in template rendering added
Actions #6

Updated by Marek Hulán about 7 years ago

Error messages could be improved by using macros, that would have explicit checks on objects. There was a discussion on foreman-dev list recently that suggested using proxy objects on which we'd define such macros. The line number is technically possible only with safe mode disabled, #18432 adds this to the log at least.

Actions

Also available in: Atom PDF