Shouldn't this be using `node['redis']['pkg_name']`?  Also, I'd say just omit the block with `action :install`.  Aaaand also, in a future revision, it's more idiomatic to have a `packages` attribute that's an array of strings.  That way, if one platform needs multiple packages you don't have to introduce more attributes.  A tiny difference, but important, I think.  Always consider the "more than one" case :smile_cat: