No need to inherit -- include and assign instead

I like this because it's more explicit and has fewer side effects. Inheriting
is good if we're using class params and want the ability to set defaults, but
we aren't and don't.
Braces around var names
Fix functionality with regular class assignment by deleting duplicate resource and adding wrapper class

Since this service shares a namevar with a service declared by the
pe_mcollective module, we can't re-declare it -- results in a dupe declaration
error. Instead, we should have a separate class for if you want to use the
module with puppet apply.
Forget about the stomp doc package on redhat/sles

Per Moses, we don't need this, and we probably don't even install it in the PE
installer, we just provide it.