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

Add IPv6 support for Debian based gmond #13

Merged
merged 5 commits into from
Apr 2, 2014
Merged

Conversation

xavpaice
Copy link

Hi,

For our environment, we use IPv6 - the addition was minimal. If you like, I could probably add that also to el5/6 but I don't have an environment in which to test that easily.

Xav Paice added 3 commits March 10, 2014 13:51
This commit adds the 'family' parameter for udp and tcp listeners
for the gmond config, for Debian based systems.
This commit adds the 'family' parameter for udp and tcp listeners
for the gmond config, for Debian based systems.
@jhoblitt
Copy link
Owner

It should be possible to test elX with the system tests. Even the old version of ganglia in EPEL5 will support setting the address family. Eg

         udp_recv_channel {
           port = 8666
           family = inet6
         }

Could you go ahead and modify the el5/6 templates to match?

For gmond.conf, add same capability as for Debian systems.
@xavpaice
Copy link
Author

Excellent thanks, I've added those in now.

Previously, the send_metadata_interval for gmond was set at 0 for
Debian based systems, and 300 for Red Hat based systems.  This change
synchronizes the setting such that Debian is now 300s as well.
@xavpaice
Copy link
Author

xavpaice commented Apr 2, 2014

Also added send_metadata_interval to 300s to the debian gmon template.

@xavpaice xavpaice closed this Apr 2, 2014
@xavpaice xavpaice reopened this Apr 2, 2014
jhoblitt pushed a commit that referenced this pull request Apr 2, 2014
Add IPv6 support for Debian based gmond
@jhoblitt jhoblitt merged commit 57fde5b into jhoblitt:master Apr 2, 2014
@jhoblitt
Copy link
Owner

jhoblitt commented Apr 2, 2014

Thank you for your work on this!

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

Successfully merging this pull request may close these issues.

2 participants