Issue #4415 has been updated by Daniel Pittman.

Chris Price wrote:
> 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.

This sort of data packing has had a long tail of trouble behind it in the past. 
 At the very least we lose a metaparameter, and we have to deal with the 
question of how that is handled in the rest of the ecosystem.  PuppetDB, 
compliance, the dashboard, and everything else would need to know that was not 
user accessible.  We would have to introduce code for "not user settable" 
metaparameters.  Hilarity ensues.

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

Regardless, I think it is unacceptable hacky, because of the long tail problems 
in the rest of the ecosystem.

We know that we need to change the catalog format to fix a number of issues, 
and are working toward that.  Until that is in place this should remain open - 
there isn't a safe half-way house to use.
----------------------------------------
Bug #4415: puppetd ignores resources in the catalog if the type code cannot be 
found
https://projects.puppetlabs.com/issues/4415#change-63448

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