On Tue, 2009-03-31 at 11:41 +1100, Avi Miller wrote:
> On Mon, Mar 30, 2009 at 11:16 PM, Bryan Kearney <bkear...@redhat.com> wrote:
> > One augeas nuance that makes it way up into puppet.... all files in
> > augeas are stored under /files. So.. you should write your command as below:
> 
> I knew this, and stared at that block for ages and still didn't spot
> that! Though, it would be nice if Puppet gave an error response. It
> appeared to have worked, but obviously didn't.

I was thinking that the provider should probably check that the context
starts with /files, but hardcoding that seems just a little icky. A
better solution would be to check that the context matches exactly one
node, and produce an error if it doesn't.

(Also, with Augeas 0.5.0, you can define variables for use in path
expressions, which should greatly reduce the need for a special context;
if that gets exposed in the type, we could deprecate the context
property. The Augeas type is definitely pushing the expressiveness of
puppet's data typing abilities)

David



--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Puppet Users" group.
To post to this group, send email to puppet-users@googlegroups.com
To unsubscribe from this group, send email to 
puppet-users+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/puppet-users?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to