Project

General

Profile

Feature #28825

Fact and report collecting should be opt-in

Added by Lukas Zapletal 12 months ago. Updated 10 months ago.

Status:
Rejected
Priority:
Normal
Assignee:
-
Category:
Facts
Target version:
-
Difficulty:
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:

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.

History

#1 Updated by Bryan Kearney 12 months ago

  • Bugzilla link set to 1793566

#2 Updated by The Foreman Bot 11 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/7475 added

#3 Updated by Lukas Zapletal 10 months 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

Also available in: Atom PDF