On Mon, Mar 22, 2010 at 9:17 PM, Luke Kanies <[email protected]> wrote: > On Mar 22, 2010, at 5:41 PM, Nigel Kersten wrote: > >> On Sun, Mar 21, 2010 at 2:59 PM, Luke Kanies <[email protected]> >> wrote: >>> >>> On Mar 20, 2010, at 8:46 AM, Brice Figureau wrote: >>> >>>> # Define a new type. >>>> def newtype(name, options = {}, &block) >>>> ... >>>> # Now set up autoload any providers that might exist for this type. >>>> klass.providerloader = Puppet::Util::Autoload.new(klass, >>>> "puppet/provider/#{klass.name.to_s}" >>>> ) >>>> >>>> # We have to load everything so that we can figure out the default >>>> type. >>>> klass.providerloader.loadall() >>>> ... >>>> end >>>> >>>> I don't think it's necessary to load the provider on the server, but >>>> this last comment looks strange to me. >>>> >>>> I think it is safe to comment those lines in puppet/metatype/manager.rb >>>> for your experiment. >>> >>> >>> My educated guess is that that call to loadall() is no longer necessary. >>> We >>> might have to modify the 'defaultprovider' method to call loadall() if >>> necessary, but it should be done on demand rather than by default. >> >> I mentioned this on IRC, but for completeness it should go here too. >> >> Looks like the problem is setting global resource type defaults. > > So that's the reason for the type getting loaded, but it's no excuse for all > of the providers getting loaded. > > Thanks for tracking that down, btw.
It wasn't much work once I noticed one of my two major client platforms didn't have this problem and the other one did :) -- You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-dev?hl=en.
