While I was going through the Configure Apache web server on Linux module on http://learn.chef.io, I noticed that the :delete action in resources/site.rb hard-codes the package name of apache2:

action :delete do
  package 'httpd' do
    action :delete
  end
end

In doing so, (1) the apache2 package will not be removed on the Ubuntu platform and (2) the :delete action might cause a chef run to fail on Ubuntu, depending on how the package resource responds to attempts to uninstall a package that doesn't even exist in repos known to the OS.

Hi!

In the "Quickstart" parts of this tutorial page:
https://learn.chef.io/modules/build-a-custom-resource/linux/virtualbox#/
the tasks are to check that this custom_web cookbook is clean.

A rule was added to foodcritic earlier this year, FC066, that is enabled by default and reports an error:

[custom_web]$ foodcritic .
Checking 4 files
x...
FC066: Ensure chef_version is set in metadata: ./metadata.rb:1

I'm not sure what the best version setting would be for this cookbook. The change was made in foodcritic here: Foodcritic/foodcritic#528

thanks!