Feature #11972
closed
Implement regex for option ignore_puppet_facts_for_provisioning
Added by Anonymous over 8 years ago.
Updated almost 6 years ago.
Description
Hey guys,
I'm currently encountering an issue, that /etc/node.rb MYNODE returns 1 because
foreman can not handle all the interfaces.
I'm running a KVM host and have around 100 tap interfaces. I don't even want them
to show up in the foreman interface.
It would be nice, to not ignore every device using the provisioning option "ignore_puppet_facts_for_provisioning=true".
Instead using a regex for that would be nice. That way, I can exclude specific interfaces.
For example an option like "ignore_puppet_facts_for_provisioning_iface=^tap[\d]+$" would be nice. That way
these interface would be excluded if matched the regex.
- Related to Bug #12013: Cannot provision host because MAC address is supposedly taken added
I believe this was solved with #12013
- Difficulty deleted (
easy)
I don't think that this was solved by #12013, this is valid RFE that would allow us to customize which interfaces should be ignored. As a part of this feature, regexp settings type would have to be added first. Also the user experience wouldn't be probably great, writing regexps is not straightforward.
Also note that #12013 ignores ^macvtap, not ^tap interfaces, so I'm afraid it does not even solve the issue descried in this Feature request.
Well, for me it would work if we just ignore ^tap devices. But probably someone will then have the issue, that he these interfaces to show up in foreman.
Best solution would be to use regexp settings type as you said.
Well we support arrays in settings today, so it might be much easier to just accept an array of prefixes that should be ignored. I know it's not as powerful as regexps but should work in most cases I think.
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/2785 added
- Pull request deleted (
- Assignee set to Marek Hulán
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
- translation missing: en.field_release set to 71
I would like to propose backporting this to the 1.10 branch if nobody sees a problem with it. While this a minor feature improvement independently, it's a major bugfix if you take #12391 into account. Since #12391 looks like it might be a little harder to solve, backporting this to 1.10 would provide a very solid workaround for the OpenStack (and other virtualization) users that depend on ignore_puppet_facts_for_provisioning to not overload Foreman with garbage nic entires. I've backported the fix to 1.10RC2 locally and it's been flawless in my testing.
- Related to Bug #12391: Discovery breaks ignore_puppet_facts_for_provisioning global setting added
- translation missing: en.field_release changed from 71 to 63
Yes, I think that's reasonable. We have an RC available so we can put it in there.
- Related to Bug #11434: Foreman 1.7.x -> 1.8.2 (Puppet collects/adds Lots of bogus interface entries) added
Also available in: Atom
PDF
Updated by 
Updated by 
Updated by 
Updated by 
Updated by