Bug #12391
closed
Discovery breaks ignore_puppet_facts_for_provisioning global setting
Description
For the last couple of versions of Foreman, the disabling of auto-creation of network interfaces has stopped working. This used to be disabled by setting ignore puppet facts for provisioning to True, which we have set, but we still get network interfaces being created. This is a big problem for us on OpenStack compute nodes, which have many virtual interfaces created, and eventually this ends up with timeouts in puppet and Foreman as there can be many hundreds of interfaces for a host, which also stops us being able to edit the host. The only fix is to delete them all via the API.
Updated by Dominic Cleal over 8 years ago
- Related to Feature #9104: Disable auto network interfaces added
Updated by Matt Jarvis over 8 years ago
In poking around in the facts importer in the codebase I spotted this commit https://github.com/theforeman/foreman/commit/379de99adbde93349824d4516b95079bfd970e90
which would partially solve the problem for us. In the meantime I've locally patched our copy to exclude the qbr|qvo|ovs|tap etc interfaces that OpenStack uses which seems to be stopping them being created.
Updated by Brandon Weeks over 8 years ago
+1 I can reproduce this.
Both ignore_puppet_facts_for_provisioning and #11972 are happening "too late." The fact_name table spam is unaffected by their use. ignore_puppet_facts_for_provisioning doesn't even seem to prevent the nics table entries from being created.
Updated by Brandon Weeks over 8 years ago
Brandon Weeks wrote:
+1 I can reproduce this.
Both ignore_puppet_facts_for_provisioning and #11972 are happening "too late." The fact_name table spam is unaffected by their use. ignore_puppet_facts_for_provisioning doesn't even seem to prevent the nics table entries from being created.
Well that's definitely the cause.
Updated by Dominic Cleal over 8 years ago
- Project changed from Foreman to Discovery
- Category changed from Importers to Discovery plugin
Ouch, that's pretty special. I'll punt this to the Discovery plugin for now then, hopefully Matt will confirm that he's got that installed too.
Updated by Matt Jarvis over 8 years ago
Yes, we've got the Discovery plugin installed.
Updated by Brandon Weeks over 8 years ago
- Related to Feature #11972: Implement regex for option ignore_puppet_facts_for_provisioning added
Updated by Brandon Weeks over 8 years ago
Thank for spotting that commit Matt, it has been a huge help for us. I've proposed backporting in #11972.
Updated by Dominic Cleal over 8 years ago
- Related to Bug #11047: Interfaces are imported even though ignore_puppet_facts_for_provisioning is set to true added
Updated by Lukas Zapletal over 8 years ago
- Subject changed from Disable auto creation of network interfaces not working to Discovery breaks ignore_puppet_facts_for_provisioning global setting
- Status changed from New to Assigned
- Assignee set to Lukas Zapletal
Yeah a bummer, did not realize this actually affect everything.
Updated by The Foreman Bot over 8 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/foreman_discovery/pull/234 added
Updated by Lukas Zapletal over 8 years ago
Matt, can you try this patch if it helps? https://github.com/theforeman/foreman_discovery/pull/234
Updated by Anonymous about 8 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset foreman_discovery|94474795d581c2703e18087004a818973f7ebb58.
Updated by Lukas Zapletal about 8 years ago
- Related to Bug #14643: Migration broken with discovered hosts added