Re: [Puppet Users] Re: hiera deep hash merges broken

2017-02-17 Thread Henrik Lindberg
On 16/02/17 15:37, Francois Lafont wrote: Hi, On 02/16/2017 09:03 PM, Henrik Lindberg wrote: We are adding support for eyaml in hiera 5 which means you will get explain support and that you can use eyaml when also using the new format. We now also test with eyaml. By the way, and just for

Re: [Puppet Users] Re: hiera deep hash merges broken

2017-02-16 Thread Francois Lafont
Hi, On 02/16/2017 09:03 PM, Henrik Lindberg wrote: > We are adding support for eyaml in hiera 5 which means you will get explain > support and that you can use eyaml when also using the new format. We now > also test with eyaml. By the way, and just for information ;), here an issue I have

Re: [Puppet Users] Re: hiera deep hash merges broken

2017-02-16 Thread Henrik Lindberg
On 16/02/17 08:16, Daniel Urist wrote: It turns out my issue was caused by eyaml no longer supporting deep hash merges. I've worked around my issue by separating out plain yaml and eyaml. Fortunately none of my existing code does a deep hash merge on the eyaml keys. It would be great to get

Re: [Puppet Users] Re: hiera deep hash merges broken

2017-02-16 Thread Daniel Urist
It turns out my issue was caused by eyaml no longer supporting deep hash merges. I've worked around my issue by separating out plain yaml and eyaml. Fortunately none of my existing code does a deep hash merge on the eyaml keys. It would be great to get some reassurance that eyaml will continue to

Re: [Puppet Users] Re: hiera deep hash merges broken

2017-02-15 Thread Francois Lafont
On 02/15/2017 09:24 PM, Eric Sorenson wrote: > That's great to hear François, thank you for testing the patch! You are welcome. :) > Our plan at > this point is to accumulate a couple more fixes and ship a new build by > Tuesday 21 Feb - > > I don't want to generate a new build containing

Re: [Puppet Users] Re: hiera deep hash merges broken

2017-02-15 Thread Eric Sorenson
That's great to hear François, thank you for testing the patch! Our plan at this point is to accumulate a couple more fixes and ship a new build by Tuesday 21 Feb - I don't want to generate a new build containing only the fix in PUP-7215 because (a) there is QA work underway on the current

Re: [Puppet Users] Re: hiera deep hash merges broken

2017-02-15 Thread Francois Lafont
On 02/15/2017 05:44 PM, Moses Mendoza wrote: > Thanks all for the reports. A fix is in progress / en route, trackable via > https://tickets.puppetlabs.com/browse/PUP-7215 Ah ok, thanks Moses for the information. I have tested in my testing VM and the commit of Thomas Hallgren seems to work

Re: [Puppet Users] Re: hiera deep hash merges broken

2017-02-15 Thread Moses Mendoza
On Wed, Feb 15, 2017 at 06:47 Francois Lafont < francois.lafont.1...@gmail.com> wrote: > Oops, I have forgotten to say this: > > On 02/14/2017 08:51 PM, Daniel Urist wrote: > > > Just found this issue, which seems to describe what's going on: > >

Re: [Puppet Users] Re: hiera deep hash merges broken

2017-02-15 Thread Francois Lafont
Oops, I have forgotten to say this: On 02/14/2017 08:51 PM, Daniel Urist wrote: > Just found this issue, which seems to describe what's going on: > https://github.com/TomPoulton/hiera-eyaml/issues/222 > So eyaml is broken wrt hash merges? Are there any workarounds available? Currently, the

Re: [Puppet Users] Re: hiera deep hash merges broken

2017-02-15 Thread Francois Lafont
Hi, On 02/14/2017 08:51 PM, Daniel Urist wrote: > Just found this issue, which seems to describe what's going on: > https://github.com/TomPoulton/hiera-eyaml/issues/222 > So eyaml is broken wrt hash merges? Are there any workarounds available? I have exactly the same problem here, and no

[Puppet Users] Re: hiera deep hash merges broken

2017-02-14 Thread Daniel Urist
Just found this issue, which seems to describe what's going on: https://github.com/TomPoulton/hiera-eyaml/issues/222 So eyaml is broken wrt hash merges? Are there any workarounds available? On Tue, Feb 14, 2017 at 12:35 PM, Daniel Urist wrote: > To get deep hash merges in