Bulk updates requires a lot of studdering about puppet-augeasproviders_shellvar HOT 6 OPEN

That could be interesting indeed. Do you want to provide a PR with such a shellvar::bulk wrapping definition?

from puppet-augeasproviders_shellvar.

I would but... sadly I don't think I know enough Ruby to add a function that does that. If I did it as a define() then it wouldn't have the performance benefit.

from puppet-augeasproviders_shellvar.

I was actually thinking of a DSL define. How would you expect to see performance benefit?

from puppet-augeasproviders_shellvar.

On the performance issue: since puppet 3.4, augeasproviders use a shared Augeas handler for all resources. Hence, Augeas is only started once, and only parses target files twice as most (one read to fill the tree, a second read to calculate the diff if changes were made, and one write to flush changes if any). All you need is to use puppet 3.4+. The only overhead to using separate resources currently is stuff done entirely in memory, so the gain wouldn't be great.

from puppet-augeasproviders_shellvar.

Oh, so the optimization is already in there. That's awesome!

I'll see if I can work up a DSL define.

from puppet-augeasproviders_shellvar.

Actually, the lib still saves the Augeas tree once per modified resource (vs once per modified property). I don't think it would be good to change this though, as that would mean that if a resource produces an invalid tree, all resources of that file will fail to apply.

from puppet-augeasproviders_shellvar.