search

chef-cookbooks / iptables

Development repository for Chef Cookbook iptables
https://supermarket.chef.io/cookbooks/iptables
Apache License 2.0
102 stars 141 forks source link

deprecated default recipe does a lot more than resources #135

Open koertkuipers opened 3 years ago

koertkuipers commented 3 years ago

the default recipe now says its deprecated, and it says we should change to using the resources.

usually when this is the case you take a look inside the default recipe and you see one or at most a few resources from the cookbook being called, e.g. the recipe can indeed be trivially replaced by the resources.

but that doesn't seem to be the case here. the recipe is doing quite a few things outside of just using the resources. so it it not clear to me how this cookbook expects me to change from the recipe to the resources. what is the suggested path here to proceed?

fozboz commented 3 years ago

All the recipe is doing is installing packages and setting up the service. Replace with iptables_packages and iptables_service resources.

bby-bishopclark commented 1 year ago

My read of koertkuipers' question is that the default recipe itself should be showing the resource usage as a transitional form. But it's not: its got a dire (misspelled) warning about deprecation and 60+ lines of cookbook.