Brad Nicholes wrote:
>>>> On 7/16/2009 at 9:10 AM, in message <[email protected]>,
>>>> Daniel
>>>>
> Pocock <[email protected]> wrote:
>
>> Brad Nicholes wrote:
>>
>>>>>> On 7/16/2009 at 8:07 AM, in message <[email protected]>,
>>>>>> Daniel
>>>>>>
>>>>>>
>>> Pocock <[email protected]> wrote:
>>>
>>>
>>>
>>>> I tried to attach this solution to the bug report, but I get this error:
>>>>
>>>> You did not enter a valid attachment number.
>>>>
>>>>
>>>> Anyhow, this is a solution for bug 232:
>>>>
>>>> http://bugzilla.ganglia.info/cgi-bin/bugzilla/show_bug.cgi?id=232
>>>>
>>>> As a consequence of applying this patch:
>>>> - whenever an RRD is created/updated, the hostname directory name will
>>>> be converted to lowercase
>>>> - any capitalization can be used with the `h' parameter to the web
>>>> interface
>>>> - whenever gmetad receives a hostname in the XML, it will use a
>>>> non-case-sensitive comparison to decide if it already has data for that
>>>> host
>>>> - the XML emitted by gmetad will show the capitalization that was
>>>> received in the XML, not the lowercase version
>>>>
>>>> Anyone applying this patch needs to rename all their hostname
>>>> directories to lowercase.
>>>>
>>>> Regards,
>>>>
>>>> Daniel
>>>>
>>>>
>>> This patch seems reasonable to me. The only part that bothers me is the
>>>
>> fact that an upgrade from a previous version might break existing installs
>> unless they rename all of their rrd directories. That could be a problem
>> for
>> some users that have a large number of monitored boxes.
>>
>>>
>>>
>> Maybe we just make it a part of trunk and the 3.2 release? People
>> (should) look more closely at the readme file when going from 3.1 to 3.2.
>>
>>
>
> I haven't actually tested the patch yet, but I'm OK with just putting it in
> trunk and not backporting it to 3.1. There was a big change between 3.0 and
> 3.1. I would expect that there would be some incompatible changes between
> 3.1 and 3.2 as well. I also think that when 3.2 is released, we should also
> have a helper script like hawson suggested in our /contrib repository. We
> just need to make sure that this in doc'ed somewhere so that when 3.2 is
> released, we can include the doc in the upgrade notes.
>
>
>
One way to make it non-disruptive for 3.1 would be making this new
behavior configurable (as I suggested in the bug) - is it worth the
extra effort of adding a config option for this, or is 3.2 intended to
be released in the new future?
Here's something that can be used as the basis for the helper script
and/or the %post section of the spec file:
killall gmetad
cd $RRDROOT
find . -type d -name '*[A-Z]*' ! -name __SummaryInfo__ -mindepth 2
-maxdepth 2 | while read ;
do
OLD_NAME=`echo "$REPLY" | cut -f3 -d/`
NEW_NAME=`echo "$OLD_NAME" | tr [A-Z] [a-z]`
CLUSTER_NAME=`echo "$REPLY" | cut -f2 -d/`
echo mv "$REPLY" "${CLUSTER_NAME}/${NEW_NAME}"
#mv "$REPLY" "${CLUSTER_NAME}/${NEW_NAME}"
done
------------------------------------------------------------------------------
Enter the BlackBerry Developer Challenge
This is your chance to win up to $100,000 in prizes! For a limited time,
vendors submitting new applications to BlackBerry App World(TM) will have
the opportunity to enter the BlackBerry Developer Challenge. See full prize
details at: http://p.sf.net/sfu/Challenge
_______________________________________________
Ganglia-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ganglia-developers
