stdlib ensure_package?

[jcpunk]

Should the package{} items be shifted to ensure_package from stdlib?

[jhoblitt]

That wouldn't hurt but using ensure_package() instead of ensure_resources() is dependent on whether or not the allow_virtual param can be removed. I believe that was added to silence a warning introduced in puppet 3.6.x, that has since been removed. Eg

puppet-ganglia/manifests/web.pp

Line 18 in 7ad4f41

allow_virtual => false,

[jhoblitt]

allow_virtual has been removed but the ensure param is now being used. A PR to convert to ensure_resources() would be welcome.