Project

General

Profile

Actions

Feature #7828

closed

Provide base URL of Foreman via foreman_url

Added by Lukas Zapletal over 9 years ago. Updated about 8 years ago.

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

Description

We currently can't use foreman_url in PXELinux Global Default or similar templates, because there is no @host accessible, therefore it leads to failed to process template: undefined method `token' for nil:NilClass errors.

We can easily allow this by skipping the token when @host attribute is not available.


Related issues 3 (1 open2 closed)

Related to Foreman - Bug #11723: Build PXE Default fails with errorClosedPhirince Philip09/08/2015Actions
Related to Foreman - Bug #14232: Missing variable to include Sat 6 Hostname in PXE MenusClosedMarek Hulán03/16/2016Actions
Related to Foreman - Feature #14245: Create new template helper setting('global_setting_name')NewLukas Zapletal03/17/2016Actions
Actions #1

Updated by The Foreman Bot over 9 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/1827 added
  • Pull request deleted ()
Actions #2

Updated by Bryan Kearney over 9 years ago

  • Bugzilla link set to 1176111
Actions #3

Updated by Dominic Cleal over 8 years ago

  • Tracker changed from Bug to Feature
  • Status changed from Ready For Testing to New
  • Assignee deleted (Lukas Zapletal)
  • Target version deleted (1.7.2)
  • Pull request added
  • Pull request deleted (https://github.com/theforeman/foreman/pull/1827)
Actions #4

Updated by Dominic Cleal over 8 years ago

  • Related to Bug #11723: Build PXE Default fails with error added
Actions #5

Updated by Dominic Cleal over 8 years ago

  • Subject changed from Allow foreman_url to be used in generic templates to Provide base URL of Foreman via foreman_url

#11723 sort of replaces this, permitting use of foreman_url in a default template. Changing this ticket to match https://github.com/theforeman/foreman/pull/1827#issuecomment-68529530 which asks for a way to have foreman_url return the base URL.

Actions #6

Updated by Marek Hulán about 8 years ago

  • Related to Bug #14232: Missing variable to include Sat 6 Hostname in PXE Menus added
Actions #7

Updated by Marek Hulán about 8 years ago

  • Status changed from New to Duplicate

The @host problem was already solved by #11723, foreman_url is still not usable with safe mode though, which should be fixed by #14232. Also instead of using foreman_url('base') we introduce new method called foreman_server_fqdn that wouldn't contain protocol. If you'd still want to add foreman_url('base') support please reopen or comment in #14232 which could probably incorporate it as well.

Actions #8

Updated by Lukas Zapletal about 8 years ago

  • Related to Feature #14245: Create new template helper setting('global_setting_name') added
Actions #9

Updated by Lukas Zapletal about 8 years ago

The new method is fine, thanks.

Actions

Also available in: Atom PDF