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

restriction of maximum clusters is only 1 ? #97

Closed
eylenth opened this issue Feb 22, 2021 · 4 comments · Fixed by #113
Closed

restriction of maximum clusters is only 1 ? #97

eylenth opened this issue Feb 22, 2021 · 4 comments · Fixed by #113
Labels

Comments

@eylenth
Copy link

eylenth commented Feb 22, 2021

why is the maximum clusters in the ganglia_validate_clusters function only be 1 ?

unless args.length == 1
raise Puppet::ParseError, "ganglia_validate_clusters(): wrong number of arguments (#{args.length}; must be 1)"
end

on our environment we have like 9 clusters configured in the /etc/ganglia/gmetad.conf configuration file.
We used an older version of the ganglia puppet module, when the ganglia_validate_clusters maximum function didn't take place.

@Debilski
Copy link
Contributor

Debilski commented Jul 8, 2021

It’s also not clear why it is declared as a Tuple[Hash]. If I change it to Array[Hash], it seems to be ok with multiple clusters.

class ganglia::gmetad (
Enum['on', 'off'] $all_trusted = 'off',
Tuple[Hash] $clusters = [{ 'name' => 'my cluster', 'address' => 'localhost' }],

@jhoblitt
Copy link
Owner

jhoblitt commented Jul 8, 2021

@eylenth #ganglia_validate_clusters should have an arity of 1 as it only works on a single array.
@Debilski Looks like that came in as part of a cleanup PR. I would be happy to merge a fix.

@eylenth
Copy link
Author

eylenth commented Jul 13, 2021

@Debilski it is working now with your PR.

@jhoblitt
Copy link
Owner

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

Successfully merging a pull request may close this issue.

3 participants