Project

General

Profile

Actions

Bug #1913

closed

Solaris architecture fact should always be hardwareisa!!!!

Added by Oliver Weinmann over 11 years ago. Updated over 11 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Host creation
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

Hi,

While testing solaris deployment on foreman 1.0.1 stable I recognized that fore some Solaris hosts the architecture is reported as sun4u and for some others it is sparc.

This is a big problem and looking at the code:

app/models/facts_importer.rb:

def architecture
      # On solaris architecture fact is harwareisa
      name = facts[:architecture] || facts[:hardwareisa]

this tells me why. If there is no good reason to use architecture on Solaris I would suggest to always use hardwareisa instead. On older versions of facter (1.5.8) on Solaris the fact architecture doesn't exist and so the correct fact hardwareisa is used. On newer versions of facter the architecture fact exists on Solaris and so foreman uses it. This is a problem when trying to deploy a new Solaris host. If the architecture is set to sun4u all the required fields are not created on the dhcp server.

Actions #1

Updated by Ohad Levy over 11 years ago

do you mind sending it as a pull request?

Actions #2

Updated by Oliver Weinmann over 11 years ago

No Problem. Just did.

Hopefully this time it is correct. :)

https://github.com/theforeman/foreman/pull/204/files

Actions #3

Updated by Oliver Weinmann over 11 years ago

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

Also available in: Atom PDF