Feature #11972
closed
Implement regex for option ignore_puppet_facts_for_provisioning
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.
Updated by Lukas Zapletal over 8 years ago
- Related to Bug #12013: Cannot provision host because MAC address is supposedly taken added
Updated by Lukas Zapletal over 8 years ago
I believe this was solved with #12013
Updated by Marek Hulán over 8 years ago
- 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.
Updated by Anonymous over 8 years ago
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.
Updated by Marek Hulán over 8 years ago
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.
Updated by The Foreman Bot over 8 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/2785 added
- Pull request deleted (
)
Updated by Marek Hulán over 8 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset 379de99adbde93349824d4516b95079bfd970e90.
Updated by Dominic Cleal over 8 years ago
- translation missing: en.field_release set to 71
Updated by Brandon Weeks over 8 years ago
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.
Updated by Brandon Weeks over 8 years ago
- Related to Bug #12391: Discovery breaks ignore_puppet_facts_for_provisioning global setting added
Updated by Dominic Cleal over 8 years ago
- 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.
Updated by Marek Hulán about 8 years ago
- Related to Bug #11434: Foreman 1.7.x -> 1.8.2 (Puppet collects/adds Lots of bogus interface entries) added