From 576b709b9b2f9ee81224f9a326020d0a4d97086b Mon Sep 17 00:00:00 2001 From: Ewoud Kohl van Wijngaarden Date: Wed, 28 Aug 2024 22:50:55 +0200 Subject: [PATCH] Disable reports by default The default value for this goes all the way back to when it was introduced in 566bf7820e56b3be503a05645cad54152309b20f. 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. --- lib/puppet/defaults.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/puppet/defaults.rb b/lib/puppet/defaults.rb index 7d41b7749e9..fd32a3e480c 100644 --- a/lib/puppet/defaults.rb +++ b/lib/puppet/defaults.rb @@ -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`.)