The module should be reworked to be forge compatible following the "puppet-module generate" skeleton.

The splunk module currently doesn't follow the emerging convention of living under the puppetlabs account and named puppetlabs-splunk

This will make it easier to find for other colleagues and the community and will facilitate topic branches and pull requests.

The splunk module currently has a large number of top level classes that must be declared in the node definition.

Experience has shown it may be more maintainable and "touchless" for the end user if there is a top level composite class in init.pp that handles declaring the other classes for the user. The user need only pass in class parameters in this way.

In addition, new features such as hiera may help make this easier and more "touchless" for the end user of the module.

The splunk module currently contains packages inside the module.

This makes the module large and more difficult to distribute. Ideally packages may be made available at yum.puppetlabs.com or apt.puppetlabs.com in the ProSvc directory with a note in the README about how to configure the repository for use with the module.