huit/puppet-ipa — pull request #49 by tmclaugh (2014-12-31 20:24:16+00:00)
Title: Service order fixes

This module suffers from a lack of anchoring to prevent resource float. The two fixes below are related to Service[ipa]. The resource was not properly anchored into ipa::server so it's realization was not bound by the requirements of the class. (ex. https://github.com/huit/puppet-ipa/blob/master/manifests/master.pp#L151) Additionally I anchored the defined types in ipa::serverinstall which was my first suspected cause. It was no until testing that where I realized the realize keyword will cause float.

I found issue similar to above regularly in this module. I'd like some direction before tackling these. Anchoring the defined types is easy. However the realize issue I'd need some input. Is there a reason packages and services are using realize at all? If I'm going to do bigger fixes I need some testing help. My use case for thos module is small and I'll probably change spots I don't use.