missing hiera lookup_options

[brightdroid]

Please add merge options, so one can add entries in all hiera layers.

Example:

You have three hiera layers (common, env, node) and you'd like to merge "systemd::loginctl_users" from env and node, it's currently not possible.

Solution

common.yaml:

# ...
lookup_options:
  systemd::loginctl_users:
    merge: deep
# ...

Docs: https://puppet.com/docs/puppet/7.6/hiera_merging.html#setting_lookup_options_to_refine_the_result_of_a_lookup

Thank you

[ekohl]

Can't you specify that in your own Hiera? I'm not sure if modules should be opinionated on it.

[kenyon]

Indeed, you can do this in your own hiera data, like I do in my common.yaml: https://github.com/kenyon/puppet/blob/0f039936c8adccdb8e8b8754a31567ee4bf05cb8/data/common.yaml#L4-L18

[brightdroid]

Sure, but I thought it might be most people need this too. And it shouldn't harm current installations.

[traylenator]

It may harm current installations if they have three layers common , env, site and they are not expecting a merge.
I think this should be closed as the May 19th suggestion is good.

[brightdroid]

Sure can be closed as I don't use the code anymore.