-
-
Notifications
You must be signed in to change notification settings - Fork 98
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
update hiera_xxx() to lookup() for hiera v5 #65
Comments
|
More specifically: - $_outputs = hiera_hash('telegraf::outputs', $outputs)
- $_inputs = hiera_hash('telegraf::inputs', $inputs)
+ $_outputs = lookup({
+ name => 'telegraf::outputs',
+ default_value => $outputs,
+ })
+ $_inputs = lookup({
+ name => 'telegraf::inputs',
+ default_value => $inputs,
+ }) |
|
You can get this behaviour by adding the variable in your lookup_options; --- # common.yaml
lookup_options:
telegraf::inputs:
merge:
strategy: deep
merge_hash_arrays: true |
hiera_hash is deprecated... lookup function needs to be used, doesn't matter the merge strategy |
|
Fixed via #82. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Using hiera 5 requires using the lookup function in place of hiera_xxx() functions. (hiera lookup docs).
Without updating this, deep merging using hiera 5 is impossible with this module, but updating will break compatibility with hiera 3.
The text was updated successfully, but these errors were encountered: