Issue #4415 has been updated by Chris Price.

Status changed from Accepted to Needs Decision
Assignee changed from Chris Price to Daniel Pittman

After a quick glance, it does look like it might be possible to solve this for 
now with a metaparameter; they do seem intact in the catalog across the wire.  
So, the idea would be:

1. Master parses manifest, recognizes that it's loading a ruby-defined type, 
and stamps a metaparameter into the catalog with a descriptive name (e.g. 
":type-loaded-from-ruby => true");
2. Agent receives catalog, tries to load type class; if it isn't able to load 
it, check for this metaparameter and fail accordingly... otherwise fall back to 
using Component like it does now.

This seems a bit hacky but I'm not 100% sure what our longer term plans are for 
extending the catalog data model... depending on the direction there, this 
might or might not be unacceptably hacky.  I'm going to put this in front of 
Daniel for an opinion.
----------------------------------------
Bug #4415: puppetd ignores resources in the catalog if the type code cannot be 
found
https://projects.puppetlabs.com/issues/4415#change-63383

Author: Dan Bode
Status: Needs Decision
Priority: Normal
Assignee: Daniel Pittman
Category: 
Target version: 3.X
Affected Puppet version: 
Keywords: pluginsync
Branch: 


site.pp
<pre>
node puppetclient {
  vcsrepo { "/usr/local/dev/repo":
    ensure => present,
    provider => git
  }
  notify { 'test123':
    require => Vcsrepo['/usr/local/dev/repo']
  }
}
</pre>
if I dont sync vcsrepo and I remove the type source code for notify.rb, then 
puppet doesn't fail and just ignores them

<pre>
debug: /Stage[main]//Node[puppetclient]/Notify[test123]/require: requires 
Vcsrepo[/usr/local/dev/repo]
info: Applying configuration version '1280516948'
debug: Finishing transaction -607606538
debug: Storing state
debug: Stored state in 0.02 seconds
notice: Finished catalog run in 0.04 seconds
debug: Using cached certificate for ca
debug: Using cached certificate for puppetclient
debug: Using cached certificate_revocation_list for ca
debug: Value of 'preferred_serialization_format' (pson) is invalid for report, 
using default (b64_zlib_yaml)
debug: report supports formats: b64_zlib_yaml marshal raw yaml; using 
b64_zlib_yaml
</pre>


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" 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-bugs?hl=en.

Reply via email to