search

sous-chefs / logrotate

Development repository for the logrotate cookbook
https://supermarket.chef.io/cookbooks/logrotate
Apache License 2.0
122 stars 152 forks source link

path property required even when the action is :disable #148

Closed kpmueller closed 3 years ago

kpmueller commented 4 years ago 
    ----------------------------------
    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 :)

bmhughes commented 3 years ago

PR coming shortly that fixes this.

kpmueller commented 3 years ago

thanks much!