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.

-- 
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.

Reply via email to