Actions
Feature #28825
closed
Fact and report collecting should be opt-in
Status:
Rejected
Priority:
Normal
Assignee:
-
Category:
Facts
Target version:
-
Difficulty:
Triaged:
Yes
Bugzilla link:
Pull request:
Description
Storing lots of fact and/or report data can quickly go out of control and our research shows that not everyone uses these Foreman features. We would like to make fact and report data storing in Foreman database opt-in via Administer setting. For new installations this will be turned off.
As part of this feature, we will modify "empty" page for Facts and Reports to ask users to enable the setting if they want to start gathering data.
For existing installations, Foreman will either keep collecting data (if there are any) or switch it off by default. It hasn't been decided yet.
Updated by
Updated by The Foreman Bot about 5 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/7475 added
Updated by Lukas Zapletal about 5 years ago
- Status changed from Ready For Testing to Rejected
The upstream patch was rejected in the form of a simple implementation of a flag that disables facts/reports parsing. Concerns:
- the patch creates problem for subscription-manager and puppet facts which we parse and store as extra host attributes like uptime or cpu/core count and memory
- there is now effort to disble or extract puppet code into plugin which will sove the issue
- we want to have less settings overall to decrease matrix of possible states for easier troubleshooting
- initial effort and evaluation has been done in order to find better ways to store reports and facts in the DB
- this is a dirty and temporary solution to the problem, we should really push for cleaner solution no matter how much more work it is
Actions