Issue #17246 has been updated by Andrew Parker. Status changed from Needs Decision to Needs More Information Assignee changed from Andrew Parker to Dominic Cleal
Dominic and I had discussed this (on #puppet-dev, I believe). We'd come to the conclusion that the feature definition needed to provide more information and consistency about when it can/should cache results. The reason for this was so that providers that depend on features being present can effectively work when features are being delivered as part of a puppet run. Dominic, can you provide a proposal of how you think the interface for defining the features in this manner might work? ---------------------------------------- Refactor #17246: Refactor Puppet::Util::Feature.add to expose caching options https://projects.puppetlabs.com/issues/17246#change-77806 Author: Dominic Cleal Status: Needs More Information Priority: Normal Assignee: Dominic Cleal Category: Target version: Affected Puppet version: 3.0.0 Keywords: features Branch: The "add" method of Puppet::Util::Feature has a couple of different caching behaviours, which should instead be explicitly exposed through the options hash or via multiple methods. Prior to #14822: * blocks get immediately run and the result cached * test results get cached on the first query to the feature After #14822: * blocks get immediately run and the result cached * tests, positive results get cached on the first query to the feature The interface should probably offer the ability to always cache, positive or negative cache and to never cache. -- 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.
