-
-
Notifications
You must be signed in to change notification settings - Fork 227
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
overwriting module params #870
Comments
you need to override $zabbix::params::zabbix_version because all class use it to customize version |
That I tried already and does not work, neither. |
can you post a manifest example of how you use the module? |
Ok, I've checked and you need to override version in all class because my suggestion wasn't applicable |
Also
|
Ops... I missed your answer. Exactly, I had to overwrite it everywhere and
could not use it in hiera. Once I'm back I can take a look at my notes, if
you need further feddback.
El dc., 27 de març 2024, 8:03, Christoph Maser ***@***.***>
va escriure:
… Also
"requirements": [
{
"name": "puppet",
"version_requirement": ">= 7.0.0 < 9.0.0"
}
],
—
Reply to this email directly, view it on GitHub
<#870 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAYX2LSUCCHKVKOGBTENN73Y2JVM7AVCNFSM6AAAAAAWLZFYNSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRSGE3DCOBRGU>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Affected Puppet, Ruby, OS and module versions/distributions
How to reproduce (e.g Puppet code you use)
among other params in hiera.
What are you seeing
not all classes are reading the value defined in hiera.
What behaviour did you expect instead
My value of zabbix_version in all the classes.
Any additional information you'd like to impart
According to https://www.puppet.com/docs/puppet/6/lang_classes.html#lang_class_define-hiera-data-class-param-defaults
params.pp should be drop in favor of hiera.
If you are maintaining older modules, you might encounter cases where class parameter defaults are set with a parameter class, such as params.pp, and class inheritance. Update such modules to use Hiera data instead. Class inheritance can have unpredictable effects and makes troubleshooting difficult. For details about updating existing params classes to Hiera data, see data in modules.
The text was updated successfully, but these errors were encountered: