Skip to content

Commit

Permalink
Set max_files to 4096 for image cache
Browse files Browse the repository at this point in the history
This should prevent the warning:

> Warning: The directory '/var/cache/smokeping/images' contains 1027 entries, which exceeds the default soft limit 1000 and may cause excessive resource consumption and degraded performance. To remove this warning set a value for `max_files` parameter or consider using an alternate method to manage large directory trees

Also see https://www.puppet.com/docs/puppet/7/types/file.html#file-attribute-max_files

We may be able to revert this in v7.0.0, which introduces an option
to not manage the image cache:

* voxpupuli#145
* voxpupuli#146
  • Loading branch information
trice-ardc committed Mar 21, 2024
1 parent c8a45fa commit 933f6fb
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions manifests/install.pp
Original file line number Diff line number Diff line change
Expand Up @@ -31,10 +31,11 @@
recurse => true,
}
file { $smokeping::path_imgcache:
ensure => directory,
owner => $smokeping::webserver_user,
group => $smokeping::webserver_group,
require => Package['smokeping'],
recurse => true,
ensure => directory,
owner => $smokeping::webserver_user,
group => $smokeping::webserver_group,
require => Package['smokeping'],
recurse => true,
max_files => 4096,
}
}

0 comments on commit 933f6fb

Please sign in to comment.