Project

General

Profile

Actions

Feature #28825

closed

Fact and report collecting should be opt-in

Added by Lukas Zapletal about 4 years ago. Updated about 4 years ago.

Status:
Rejected
Priority:
Normal
Assignee:
-
Category:
Facts
Target version:
-
Difficulty:
Triaged:
Yes
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.

Actions #1

Updated by Bryan Kearney about 4 years ago

  • Bugzilla link set to 1793566
Actions #2

Updated by The Foreman Bot about 4 years ago

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

Updated by Lukas Zapletal about 4 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

Also available in: Atom PDF