[sr-dev] Pipelimit/ratelimit

[Daniel-Constantin Mierla]

On 01/07/14 16:58, Olle E. Johansson wrote:
> On 01 Jul 2014, at 13:46, Olle E. Johansson <oej at edvina.net> wrote:
>
>> Hi!
>>
>> These modules seems to depend a bit on a Linux system, reading /proc data, but it's not documented. Should we add this to the docs? Does the module work on *BSD systems at all?
>>
>> The database support in pipelimit seems like a good idea, but there's no reload RPC command. Is there a reason for this? WIll it not work if we implement it?
>>
>> I would like to be able to change the pipes during runtime.
> Running on my dear old Mac:
>
>   2(7798) ERROR: ratelimit [ratelimit.c:306]: get_cpuload(): could not open /proc/stat
>   2(7798) ERROR: <core> [statistics.c:390]: get_used_waiting_queue(): Could not open /proc/net/udp. kamailioMsgQueueDepth and its related alarms will not be available.
>   2(7798) ERROR: <core> [statistics.c:390]: get_used_waiting_queue(): Could not open /proc/net/tcp. kamailioMsgQueueDepth and its related alarms will not be available.
>   2(7798) ERROR: <core> [statistics.c:390]: get_used_waiting_queue(): Could not open /proc/net/tcp. kamailioMsgQueueDepth and its related alarms will not be available.
>   2(7798) ERROR: <core> [statistics.c:390]: get_used_waiting_queue(): Could not open /proc/net/udp. kamailioMsgQueueDepth and its related alarms will not be available.
>   2(7798) ERROR: <core> [statistics.c:390]: get_used_waiting_queue(): Could not open /proc/net/tcp. kamailioMsgQueueDepth and its related alarms will not be available.
>   2(7798) ERROR: <core> [statistics.c:390]: get_used_waiting_queue(): Could not open /proc/net/tcp. kamailioMsgQueueDepth and its related alarms will not be available.
>
> These get repeated for every timer run. The source code seems to indicate that someone worked
> on a way to push this information over the ctl connection.
At least should be made optional, in order to have the other algorithms 
working and the docs updated to reflect this.

Cheers,
Daniel

-- 
Daniel-Constantin Mierla - http://www.asipto.com
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda