Project

General

Profile

Bug #16073

Changing interfaces on unmanaged hosts leads to an invalid host

Added by Justin Sherrill almost 4 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Normal
Category:
Facts
Target version:
Difficulty:
easy
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

If an unmanaged host has any interface mac address changed, either because it was re-provisioned externally, or some other external condition occurred, facts being uploaded with the new mac address in will result in a duplicate interface being created.

For example if a host has an interface 'eth0' with mac address 'foo' and is currently uploading facts to foreman but its 'eth0' network device changes mac address to 'bar', a second 'eth0' interface will be created.

If this occurs the host is left in an invalid state and requires the old network interface to be removed (in fact even trying to correct the situation via the UI leads to issues as it doesn't seem to be possible to delete the old primary interface in this case. At least not very easily).


Related issues

Related to Foreman - Bug #16547: Fact importing fails with NoMethodError: undefined method `first' for nil:NilClassClosed2016-09-14

Associated revisions

Revision 7b75a6a0 (diff)
Added by Justin Sherrill almost 4 years ago

Fixes #16073 - match up nic identifiers for unmanaged hosts

History

#1 Updated by The Foreman Bot almost 4 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/3723 added

#2 Updated by Justin Sherrill almost 4 years ago

  • Bugzilla link set to 1363749

#3 Updated by Marek Hulán almost 4 years ago

  • Legacy Backlogs Release (now unused) set to 160

#4 Updated by Justin Sherrill almost 4 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#5 Updated by Dominic Cleal almost 4 years ago

  • Related to Bug #16547: Fact importing fails with NoMethodError: undefined method `first' for nil:NilClass added

Also available in: Atom PDF