Issue #7959 has been updated by Dominic Cleal.

Jacob Helwig wrote:
> Dominic: I've seen suggestions that people use the lens/incl attributes come 
> up fairly frequently recently.  Should these be made required rather than 
> optional?  I'm afraid I don't actually know that much about Augeas, but it 
> seems like if it's something that comes up as frequently as it has been then 
> we should just be requiring it.

I don't think so.  It's only needed for two reasons:

1. Supporting non-standard file locations or lenses
1. Performance, as in this case: instead of Augeas loading every file it knows, 
you restrict it to speed up execution.  Two other issues help this enormously, 
#2728 doubles speed when a change has to be made and #7285 would also make a 
big difference (another 50% for me).

I think the performance fixes will help the occasional query like this, and 
other queries are expected use cases for lens+incl.  To make it mandatory would 
make specifying and using the type much harder for the large majority of cases 
with little benefit other than speed.

It also requires that the user work out which lens is used for their particular 
file, which requires some code diving or grepping.
----------------------------------------
Bug #7959: augeas type_check RAM usage
https://projects.puppetlabs.com/issues/7959

Author: Markus Falb
Status: Investigating
Priority: Normal
Assignee: Jacob Helwig
Category: agent
Target version: 
Affected Puppet version: 
Keywords: augeas
Branch: 


    augeas { "ram fresser":
        type_check => true,
        context => "/files/etc/sysctl.conf",
        changes => "set net.ipv4.ip_forward 1",
    }

and puppet needs 1GB RAM ? This looks wrong to me. I only see this behaviour if 
type_check is true.


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