sous-chefs / logrotate

Development repository for the logrotate cookbook

Home Page:https://supermarket.chef.io/cookbooks/logrotate

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

path property required even when the action is :disable

kpmueller opened this issue · comments

    ----------------------------------
    path is a required property

    Resource Declaration:
    ---------------------
    # In /var/chef/cache/cookbooks/piazza_app/recipes/logs.rb

     32: logrotate_app 'petty_gc' do
     33:   action :disable
     34: end
     35:

I have a resource declaration to remove an old log rotation. It does not have a path, since it only needs to remove a chef log rotation by Chef name (I assume?). I don't know if this is something you want to fix or not, but I thought i would flag it.

It may be an artifact of Chef 16+ where required properties are actually required, now :)

PR coming shortly that fixes this.

thanks much!