Feature #16890

Prevent host discovery for existing MAC/IP addresses

Added by Andrew Blatt 9 months ago. Updated 4 months ago.

Assigned To:Lukas Zapletal
Category:Discovery plugin
Target version:Plugin 8.0.1
Difficulty: Pull request:https://github.com/theforeman/foreman_discovery/pull/324
Bugzilla link:
Story points-
Velocity based estimate-


Ran into a situation where someone deleted the pxe/macfile from tftpserver. This allowed server to use default pxe file that says to boot into discovery image. When discovery image boots, it should check if ip/mac already exists in host table, and if so, don't add into discovered_hosts table. When this occurs and I remove the entry from discovered_hosts, the existing host entry loses its FACTs information.

Related issues

Related to Discovery - Bug #17260: Discovered (7.0) host does not delete with error "undefin... Closed 11/07/2016

Associated revisions

Revision 210f143b
Added by Lukas Zapletal 5 months ago

Fixes #16890 - prevent discovery of managed hosts


#1 Updated by Dominic Cleal 9 months ago

  • Project changed from Foreman to Discovery
  • Category deleted (Inventory)

#2 Updated by Lukas Zapletal 8 months ago

  • Subject changed from Prevent host booting into discovery from registration if server already exists in host table. to Prevent host discovery for existing MAC/IP addresses
  • Category set to Discovery plugin
  • Priority changed from Normal to Low

Well this is quite special case when someone delete file on TFTP server. If this gets ever implemented, this must be an opt-in setting (each MAC and IP). Only provisioning NICs must be matched as well.

#3 Updated by Lukas Zapletal 6 months ago

  • Related to Bug #17260: Discovered (7.0) host does not delete with error "undefined method `operatingsystem'" added

#4 Updated by Lukas Zapletal 6 months ago

Experienced this today, when existing (managed) host does exist and it boots into Discovery mode, existing host is "converted" into discovered host, facts are deleted. But what's more important, the host is in error state and cannot be deleted (workaround is via rake console). I will make a patch to prevent to discover an existing server.

#5 Updated by The Foreman Bot 6 months ago

  • Status changed from New to Ready For Testing
  • Assigned To set to Lukas Zapletal
  • Pull request https://github.com/theforeman/foreman_discovery/pull/324 added

#6 Updated by Anonymous 5 months ago

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

#7 Updated by Lukas Zapletal 4 months ago

  • Target version set to Plugin 8.0.1

Also available in: Atom PDF