[collectd] Patch: Support custom CPU interval v2

[Boian Berberov]

I have this in my configuration file:

<LoadPlugin cpu>
        Interval 5
</LoadPlugin>

When I start collectd I get:

Ignoring unknown LoadPlugin option "Interval" for plugin "cpu"

Is this what you are trying to enable with this patch?



On Tue, Mar 11, 2014 at 3:18 AM, Lee Hardy <lee at leeh.co.uk> wrote:

> Ah, I started with auto loading plugins and must've completely missed that
> when I dropped back to manually setting them.
>
> That's working thanks.
>
> Cheers,
>
>
>
>
> On 11 March 2014 08:03, Yves Mettier <ymettier at free.fr> wrote:
>
>> Hello,
>>
>> Don't you mean this ?
>>
>> Interval 15
>> <LoadPlugin memory>
>>     Interval 10
>> </LoadPlugin>
>> <Plugin memory>
>> </Plugin>
>>
>> More here : http://collectd.org/documentation/manpages/
>> collectd.conf.5.shtml#global_options
>>
>> Regards,
>> Yves
>>
>> Le 2014-03-11 08:43, Lee Hardy a écrit :
>>
>>  It doesn't work, if I try doing this for the memory plugin (which I
>>> haven't
>>> patched) with the following config:
>>>
>>> Interval 15
>>> LoadPlugin memory
>>> <Plugin "memory">
>>>       Interval 10
>>> </Plugin>
>>>
>>> This throws the following error:
>>> Mar 11 07:28:55 rpm-i386 collectd[1196]: Found a configuration for the
>>> `memory' plugin, but the plugin isn't loaded or didn't register a
>>> configuration callback.
>>>
>>> It then logs out at 15s intervals rather than the 10s.
>>>
>>>
>>> Is there a different approach to the fix I should be taking?  I don't
>>> mind
>>> taking another look..
>>>
>>> Cheers,
>>> Lee H
>>>
>>>
>>> On 10 March 2014 23:58, Manuel Luis Sanmartín Rozada
>>> <manuel.luis at gmail.com>wrote:
>>>
>>>  Hello.
>>>>
>>>> If I understand you problem correctly,  you can use a function that
>>>> collectd already have:
>>>> set a plugin-specific interval for collecting metrics:
>>>>
>>>>             <LoadPlugin cpu>
>>>>               Interval 60
>>>>             </LoadPlugin>
>>>>
>>>> Or am I missing something?
>>>>
>>>>
>>>> On Sun, Mar 9, 2014 at 6:47 PM, Lee Hardy <lee at leeh.co.uk> wrote:
>>>>
>>>>  Hey,
>>>>>
>>>>> I've updated the CPU interval patch to make it more generic, by adding
>>>>> a
>>>>> plugin_update_interval() function called as part of the config
>>>>> function so
>>>>> I can update other plugins more easily.
>>>>>
>>>>> When the aggregation module is enabled for CPU information though, it
>>>>> seems to be throwing an error:
>>>>> Mar 9 17:22:36 rpm-i386 collectd[26640]: plugin_value_list_clone:
>>>>> Unable
>>>>> to determine interval from context for value list
>>>>> "rpm-i386.build.local/aggregation-cpu-average/cpu-idle". This
>>>>> indicates a
>>>>> broken plugin. Please report this problem to the collectd mailing list
>>>>> or
>>>>> at <http://collectd.org/bugs/>.
>>>>>
>>>>> This seems to happen even when I specify the interval via
>>>>> plugin_register_complex_read() as well, so I'm not entirely sure if
>>>>> I've
>>>>> missed something or there is some issue with the aggregation module.
>>>>>
>>>>> Anyone have any ideas?
>>>>>
>>>>> Cheers,
>>>>> Lee H
>>>>>
>>>>> _______________________________________________
>>>>> collectd mailing list
>>>>> collectd at verplant.org
>>>>> http://mailman.verplant.org/listinfo/collectd
>>>>>
>>>>>
>>>>>
>>>>
>>> _______________________________________________
>>> collectd mailing list
>>> collectd at verplant.org
>>> http://mailman.verplant.org/listinfo/collectd
>>>
>>
>> _______________________________________________
>> collectd mailing list
>> collectd at verplant.org
>> http://mailman.verplant.org/listinfo/collectd
>>
>
>
> _______________________________________________
> collectd mailing list
> collectd at verplant.org
> http://mailman.verplant.org/listinfo/collectd
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.verplant.org/pipermail/collectd/attachments/20140325/1dd1a382/attachment.html>