search

hartmantis / clamav-chef

A Chef cookbook for the Clam AntiVirus application
11 stars 57 forks source link

Make freshclam execution optionnable via attribute (even of debian/ubuntu) #52

Closed lotooo closed 8 years ago

lotooo commented 8 years ago

It is currently possible to use freshclam as a service the attribute node['clamav']['freshclam']['enabled'], which is great.

But if you are running a VM with paltform_family == 'debian' a freshclam execution is forced.

  if !node['clamav']['freshclam']['enabled'] || platform_family == 'debian'
    notifies :run, 'execute[freshclam]', :immediately
  end

Is there a reason why this is not 100% customizable via the attribute ? (something like default is true but if you are running debian the default is false). That way we can choose to force a freshclam refresh during the chef_run or not.

lotooo commented 8 years ago

The goal is to avoid a long chef_run not to avoid the refresh, move this to :delayed instead of :immediatly should be enough.