>>> On 9/9/2008 at 3:24 PM, in message <[EMAIL PROTECTED]>, <[EMAIL PROTECTED]> wrote:
> - * gmond/gmetad: The gmetad code can not handle a host with no associated > + * libganglia: The gmetad code can not handle a host with no associated > cluster, therefore the gmond code must always include a cluster XML > tag. > This patch matches up the default values for the cluster section of > gmond with the default gmond.conf so that a cluster name will always > @@ -182,6 +175,10 @@ > http://ganglia.svn.sourceforge.net/viewvc/ganglia?view=rev&revision=1712 > +1: bnicholes > +1: hawson > + -1: carenas > + carenas: not having a cluster section is documented as valid and might > + be a common setup with 2.5.x (still the official ganglia in > + Debian and [Open]SuSE) > Carlo, I don't understand the -1 vote on this. This fixes an issue that currently exists in all previous releases by simply making sure that a cluster name value exists rather than a NULL value. A NULL value for a cluster name is invalid as shown by the code itself so the configuration should have never allowed the cluster name to be NULL. This is the bug that the patch is fixing. It also syncs up the internal default cluster values with the values that are shown in the default configuration file. This has nothing to do with whether or not version 2.5.x actually produces or doesn't produce a cluster tag. If the cluster section is commented out of version 2.5.x, it will probably segfault as well. As a side note, if the documentation or the DTD states that a missing cluster section is valid, it is the documentation that has a bug. The code does not handle a missing cluster section and never has even though a mis-configuration of the cluster section allowed it. It is certa inly not worth the effort to implement functionality that is only produced by a mis-configuration of gmond. It is better to prevent the mis-configuration in the first place. Brad ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ Ganglia-developers mailing list Ganglia-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ganglia-developers