Issue #4415 has been updated by Chris Price.
I generally agree with this assessment... however, I suspect that our solution for making the catalog more generically extensible is going to be to add yet another bucket that we can dump metadata into. So we'll have "parameters", (the not-so-fantastically-named) "meta-parameters", and then something like "system-parameters" or "puppet-parameters" or "internal-parameters"... this new addition being basically Just Another Hash that we can use to contain key/value pairs that allow us to more intelligently distinguish between and handle various resources. When it comes down to it, it seems like the implementation will be so similar to the existing metaparameters that it is a bit of a shame to have to leave a fairly hideous bug like this open for a long period of time waiting for that to become available. That said, I would probably make the same call (avoiding hackery). Just wanted to get the option documented so that the cost/benefit can be weighed against the longevity of bugs like this in the event that we don't get the catalog extensibility issue resolved soon. ---------------------------------------- Bug #4415: puppetd ignores resources in the catalog if the type code cannot be found https://projects.puppetlabs.com/issues/4415#change-63476 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.
