-
Notifications
You must be signed in to change notification settings - Fork 135
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
validate_legacy() is deprecated from stdlib #879
Comments
Same problem |
Unfortunately, I have a feeling they absolutely do not care about issues created by anyone but themselves. Given the complete silent treatment, we're working on our own branch: https://github.com/cusae/wazuh-puppet/tree/v4.7.2_cusae And I don't think we'll be trying to push anything anymore. The most recent version of the module even embark another module hijacking, see #1038 ; and it does not seem any "official" are worried about that. Something has gone very wrong here. |
There hasn't been a version of the puppet module that we can use. I'm not sure what is going on, but it seems like they don't have any idea how to write puppet code. |
Hello,
puppetlabs-stdlib deprecate'd the validate_legacy in 9.0.0 back in May. Thus, the current wazuh-puppet 4.7.1 produces an error when using it with a >=9.0.0 stdlib:
Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Error while evaluating a Function Call, deprecation. validate_legacy. This method is deprecated, please use Puppet data types to validate parameters at ["/etc/puppetlabs/code/environments/production/modules/wazuh/manifests/agent.pp", 255] (file: /etc/puppetlabs/code/environments/production/modules/wazuh/manifests/agent.pp, line: 255, column: 3) on node XXX
It's used quite a few times in the source:
wazuh-puppet $ grep -r validate_legacy | wc -l
17
Reference:
puppetlabs/puppetlabs-stdlib#1352
It could probably be replace with is_a:
https://github.com/puppetlabs/puppetlabs-stdlib/blob/main/REFERENCE.md#is_a
The text was updated successfully, but these errors were encountered: