Feature #9490
closedSettings -> Discovered provide setting to specify fact to use for hostname
Description
Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1186537
Description of problem:
RFE
Would like to be able to set the fact that is used for the hostname of the system. e.g. uuid, asset_tag, serialnumber, etc.. (or combination of tags.. nice to have.)
By default this is set in /opt/rh/../models/hosts/discovered.rb as:
fact_name = Setting[:discovery_fact] || 'macaddress'
hostname = facts[fact_name].try(:downcase).try(:gsub,/:/,'').try(:sub,/^/,'mac')
Issue is changing this fact changes the population of the macaddress on provisioning as well. Hostname setting should really be separate.
Version-Release number of selected component (if applicable): 6.0.7
How reproducible: Always
Steps to Reproduce:
1. Set discovery_fact to serialnumber
2. Serialnumber gets substituted for macaddress when provisioning.
3.
Actual results:
as above
Expected results:
have separate setting that allows these two values to be independent of one another.
Additional info:
Updated by Lukas Zapletal almost 10 years ago
- Assignee deleted (
Lukas Zapletal)
Let's do this for Discovery 2.1 release. Easy enough, note there is implementation patch in the downstream BZ.
Updated by Lukas Zapletal almost 10 years ago
- Related to Tracker #9018: Foreman Discovery 2.1 features added
Updated by Lukas Zapletal almost 10 years ago
- Category set to Discovery plugin
- Target version set to Discovery Plugin 3.0.0
Updated by Lukas Zapletal almost 10 years ago
- Related to deleted (Tracker #9018: Foreman Discovery 2.1 features)
Updated by Ori Rabin almost 10 years ago
- Status changed from New to Assigned
- Assignee set to Ori Rabin
Updated by Lukas Zapletal almost 10 years ago
- Target version changed from Discovery Plugin 3.0.0 to Discovery Plugin 4.0.0
Updated by Lukas Zapletal over 9 years ago
- Target version deleted (
Discovery Plugin 4.0.0)
Ori what is the status? I am pulling this off 4.0 version.
Updated by The Foreman Bot over 9 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/foreman_discovery/pull/213 added
- Pull request deleted (
)
Updated by The Foreman Bot almost 9 years ago
- Pull request https://github.com/theforeman/foreman_discovery/pull/246 added
Updated by Imri Zvik almost 9 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset foreman_discovery|1f4e985b05d4af9d05e361a0fa9bdcbea3bc8efc.