Skip to content

Commit

Permalink
Disable reports by default
Browse files Browse the repository at this point in the history
The default value for this goes all the way back to when it was
introduced in 566bf78. I'd argue it a
bad default, because it just writes files without any clean up process
in place. That only causes hard disks to fill up.

Having reports should be an opt in feature, where they make a careful
choice. PuppetDB is an obvious recommended option but not one that can
be relied on by default.
  • Loading branch information
ekohl committed Aug 28, 2024
1 parent 267684a commit 576b709
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/puppet/defaults.rb
Original file line number Diff line number Diff line change
Expand Up @@ -1477,7 +1477,7 @@ def self.initialize_default_settings!(settings)
:mode => "0750",
:desc => "The directory in which serialized data is stored, usually in a subdirectory."},
:reports => {
:default => "store",
:default => "none",
:desc => "The list of report handlers to use. When using multiple report handlers,
their names should be comma-separated, with whitespace allowed. (For example,
`reports = http, store`.)
Expand Down

0 comments on commit 576b709

Please sign in to comment.