I recently deployed the CollectM project (https://github.com/perfwatcher/collectm), which embeds your library, on our Windows servers. I found that after a period of time ranging from minutes to hours, the remote collectd server being written to would start consuming all available memory before finally being nuked by the OOM killer. Which is obviously a horrible bug on the main collectd project's part, but I wanted to see what was different about the traffic coming from node-collectdout to trigger it.

The remote server is currently running collectd 5.4.2 and has been handling about 600 Linux clients with no issue for years. As soon as we added the Windows client (using node-collectdout), the problem surfaced. I disabled the 15 or so Windows clients, and it immediately stabilized.

I captured the traffic between the collectd server and a Windows client, and Wireshark's collectd parser flagged every packet as invalid. Due to a bunch of 0x00 bytes at the end. Looking at the code, it sends a 1024 byte payload every time, regardless of the actual length of the data. A bunch of zeroes are sent after the end of the data to fill out the packet.

I modified the code to match the packet size to the data. Since deploying that patch, the server has been running with zero issues. I'll submit a pull request shortly. nodejs is definitely not my "native language" so I'm open to suggestions on how to do this differently.

Even if collectd fixes their server to handle this better, it's still preferable not to send hundreds of bytes of pointless zeroes. Not a big deal in the era of 10 gig ethernet, but still :)

It would be great to be able to send notification through node-collectdout

Hello:

It is not clear to me if you are taking into consideration Number.MAX_VALUE for counters.
I mean, if I'm counting values for a long period of time, I might get to an invalid value for a javascript number, so I'd need to revert counter to 0.

This is the code for addCounter function:

addCounter: function(name, instance, increments) {
    if (increments.constructor !== Array)
        increments = [increments];

    doubleHashUpdate(this.counters, name, instance, function(counters) {
        return increments.map(function(increment) {
        var counter = counters && counters[0] || 0;
        return counter + increment;
        });
    });
    }

Where do you store the result of "counter + increment" of the map function?

Thanks

Hi,

I have an issue with this code :

var Collectd = require('collectdout');
client = new Collectd(1000, 'localhost', 25826);
var plugin = client.plugin('df', 'c');
plugin.setGauge('df', '', [10, 10]);

Collectd say : ERROR: Numerical result out of range

An idea ?

Hi! This is a pretty interesting feature, but couldn't find docs for it.
Do you mind sharing more?