meh. that was a copy/paste bug. of course you are right with auto-require.

However concerning subscribe or not, imho thats exactly what puppet is for: rolling out changes and taking care of whatever needs to be restarted. If tested properly, automated restarts shouldnt be a problem.

But I can also understand your POV so we dont really need to discuss that :) I'll just leave the subscribe out then as the rest of the module also doenst have that and probably never will.