Skip to content
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

Warning message from config.pp #575

Open
hawkeye-7 opened this issue Aug 28, 2023 · 0 comments
Open

Warning message from config.pp #575

hawkeye-7 opened this issue Aug 28, 2023 · 0 comments

Comments

@hawkeye-7
Copy link

Describe the Bug

A warning message is emitted:

Warning: This function is deprecated, please use stdlib::merge instead. at ["/etc/puppetlabs/code/environments/esgfv4/r10k/modules/haproxy/manifests/config.pp", 23]:
(location: /etc/puppetlabs/code/environments/esgfv4/r10k/modules/stdlib/lib/puppet/functions/deprecation.rb:35:in `deprecation')

Expected Behavior

No warning or error messages!

Steps to Reproduce

puppet apply with puppetlabs::haproxy installed

Environment

  • Version 7.2.0

Additional Context

The problem is the use of merge instead of stdlib::merge. For backwards compatibility, I have created a version that uses the plus sign instead:

23,24c23,24
< $_global_options = $haproxy::params::global_options + $global_options
< $_defaults_options = $haproxy::params::defaults_options + $defaults_options

$_global_options   = merge($haproxy::params::global_options, $global_options)
$_defaults_options = merge($haproxy::params::defaults_options, $defaults_options)

This version can be found at https://github.com/hawkeye-7/puppetlabs-haproxy.git

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants