Issue #4433 has been updated by Nigel Kersten.
R.I. Pienaar wrote: > Your get() function has more or less done a clean room implementation of > extlookup and come to the conclusion that the feature set is whats needed, so > why not work on whats there already by just making it pluggable? Because: a) it doesn't offer enough over extlookup b) it fails to solve the use case of allowing all data for a customer/datacenter/custom bucket to be contained in one location. The actual conclusion I came to was that it's sadly lacking. Diving into your latest implementation today, I encourage others to have a look at it as well. http://www.devco.net/archives/2011/05/28/pluggable_extlookup_for_puppet.php ---------------------------------------- Feature #4433: Add hash support to extlookup https://projects.puppetlabs.com/issues/4433 Author: R.I. Pienaar Status: Needs Decision Priority: Normal Assignee: Nigel Kersten Category: functions Target version: 2.7.x Affected Puppet version: 2.6.0 Keywords: Branch: http://github.com/ripienaar/puppet/tree/extlookup_hash_support I've added hash support to extlookup. Really it's time to add yaml support but thats a whole different story. CSV files can now have: <pre> test,key=val another,key=val,foo=bar </pre> And it will return hashesh: <pre> {"key" => "val"} {"key" => "val", "foo" => "bar"} </pre> In the event that not all values have a = in them it will just behave as old and return arrays. My concern here is that we have a small case of breaking backward compatibility should someone already have key=val style values. We should make this clear in the release notes. -- 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.
