Array, Boolean and other types support in "hostgroup parameters"
|Status:||Ready For Testing|
|Assigned To:||Kavita Gaikwad|
|Target version:||Team Anurag Iteration 22|
|Found in release:||Pull request:||https://github.com/theforeman/foreman/pull/5241|
|Velocity based estimate||-|
Currently "string" is the only parameter type supported in hostgroup parameters on foreman 1.3. Our customers desperately need to put Array and Boolean type of parameters (possibly other types as well) into the hostgroup for better puppet parameter management of they application clusters. Since multiple parameter types are supported in Smart Variables, why the hostgroup parameters (as well host parameter) have to stuck with just one type (string).
#1 Updated by Dominic Cleal over 4 years ago
- Release deleted (
We have talked about replacing general parameters with the smart variables implementation at some point, but obviously removing the association to Puppet classes. It might not be exactly what you need, but it is possible use smart class parameters and override them from the host group and host edit pages.
#2 Updated by Alejandro Falcon over 4 years ago
I was going to open a new issue about this when I found this one. I like using smart class parameters as you mention but I use Example42 modules which have a nice feature to keep the original modules up to date and it is a my_class parameter on each module that allow me to add a desired custom class as a parameter in foreman and obviously I can't use smart class parameters with them so I have to use global parameters for these custom classes. It would be great to have all data types working on global params as in smart class parameters.
#3 Updated by Neil Miao over 4 years ago
ideally, we should use smart class parameters, but the reality is that
we have too many legacy home-baked modules (hundreds, if not
thousands) lying in our puppet code-based that it might take us months
to refactor them to the proper parameterized classes.
I certainly hope adding this feature to Foreman is quicker than that.