[Pkg-puppet-devel] Bug#1070744: Bug#1070744: /usr/bin/puppet: puts non-regeneratable data in /var/cache

[Jérôme Charaoui]

Hello,

Le 2024-10-24 à 05 h 18, Hendrik Jaeger a écrit :
> According to upstream, the vardir is actually for caching stuff, so IMHO generally vardir being in /var/cache is not wrong.
> What I consider wrong, though, is putting reports there, as they cannot be regenerated which is what the FHS demands on data put there.
> And this should probably be forwarded upstream.
> (The same may be said about other things, like `bucketdir` and `clientbucketdir`, neither of which can be regenerated. But that should be a separate bug to not inflate this one!)
> 
> To make a concrete suggestion:
> set reportdir = $logdir/reports
> 
> What do you think?

I think it would make sense, and your observations about the bucket 
directories are also relevant.

... but I'm still hesitant to carry this as an additional patch in 
Debian: it really should be implemented upstream, since the FHS is not 
Debian-specific at all.

Plus, if implemented, what do we do with the data in the previous path? 
Forcibly move it? Ignore it? Prompt the user?

Worth noting is that since the last puppet-agent upload, the reports 
feature now defaults to disabled (no reports generated).

-- Jérôme