Bug#930058: unblock: puppet/5.5.10-3

[Thomas Goirand <zigo@debian.org>]

On 6/15/19 7:54 PM, Paul Gevers wrote:
> Control: tags -1 moreinfo
> 
> Hi Thomas,
> 
> On 06-06-2019 10:36, Thomas Goirand wrote:
>> Version 5.5.10-3 adds a tiny cron.daily job which cleans-up the
>> /var/lib/puppet/reports folder to avoid that a puppet-master
>> server gets its HDD full, which potentially could be very harmful
>> for a deployment.
> 
> This seems slightly controversial to me (as hinted by a comment in the
> bug as well). Don't you think this warrants a note in NEWS?
> 
> Paul

Hi,

Thanks for your time reviewing this yet another unblock request.

I don't think people read more the NEWS than the changelog. Respectfully
(please don't see any aggressiveness here: there's none), if you really
want to waste everyone's time on this, then I can do it, but I don't
think this helps. There's not much time before the final release.

Having run puppet on a small virtual machine with 40 GB disk, and having
it taken down by 800 MB reports every day for every compute in the
cluster, no, this isn't controversial. This is completely mandatory.
What I believe puppet user do is set this up by themselves, or disable
reports all together. Keeping 1 month of reports is very conservative.

Cheers,

Thomas Goirand (zigo)