Project

General

Profile

Actions

Bug #38688

closed

Provisioning templates use host_enc macro to determine the realm

Added by Ewoud Kohl van Wijngaarden 5 months ago. Updated 5 months ago.

Status:
Closed
Priority:
Normal
Category:
Unattended installations
Target version:
-
Fixed in Releases:
Found in Releases:

Description

The provisioning templates use some variation of:

if host_enc['parameters']['realm'] && @host.realm

This requires the full ENC to be rendered for no good reason: the realm parameter is equivalent to @host.realm&.name (https://github.com/theforeman/foreman/blob/c28ab6ff1cd0b8fe77b853a5bfbd70ef86508af9/app/models/host_info_providers/static_info.rb#L54-L55). Since name is mandatory and we already check for @host.realm anyway we can drop the redundant check.

Actions #1

Updated by Ewoud Kohl van Wijngaarden 5 months ago

  • Description updated (diff)
Actions #2

Updated by The Foreman Bot 5 months ago

  • Status changed from New to Ready For Testing
  • Assignee set to Ewoud Kohl van Wijngaarden
  • Pull request https://github.com/theforeman/foreman/pull/10659 added
Actions #3

Updated by The Foreman Bot 5 months ago

  • Fixed in Releases 3.17.0 added
Actions #4

Updated by Ewoud Kohl van Wijngaarden 5 months ago

  • Status changed from Ready For Testing to Closed
Actions #5

Updated by The Foreman Bot 5 months ago

  • Pull request https://github.com/theforeman/foreman/pull/10670 added
Actions #6

Updated by The Foreman Bot 5 months ago

  • Fixed in Releases 3.16.0 added
Actions

Also available in: Atom PDF