Project

General

Profile

Bug #1323

Operating system should not be required if host is unmanaged

Added by Greg Sutcliffe over 7 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Normal
Category:
Host creation
Target version:
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

If I have a host with no Operating System set, and I have Unattended => true in my Settings, then Foreman will refuse to display the host, because it tries to render the template box for the bottom-right corner of the host page. Instead, you get a error stating "Operating system must be set"

I believe it makes more sense to render that template if the host is managed - just because you want to do unattended installations for some hosts doesn't mean you want to do it for all hosts...

This patch solves the problem for my use case, but there may be a better way to solve it...

templates.diff templates.diff 509 Bytes Patch to fix Greg Sutcliffe, 11/15/2011 06:30 AM

Associated revisions

Revision 4aac99c0 (diff)
Added by Greg Sutcliffe over 7 years ago

fixes #1323 - Check host.managed before rendering templates on host page

Signed-off-by: Greg Sutcliffe <>

History

#1 Updated by Ohad Levy over 7 years ago

  • Category set to Host creation
  • Assignee set to Greg Sutcliffe
  • Target version set to 0.4.1

the patch should be a combination of both conditions, can you please send it as a git patch?

thanks

#2 Updated by Anonymous over 7 years ago

  • Status changed from New to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF