Fix for sample munin plugin script #3

Merged
merged 1 commit into from May 12, 2012

Projects

None yet

2 participants

@rstml
Contributor
rstml commented May 8, 2012

adds support for multiple JMX source caching on a single node and fixes bug with ambiguous keys (e.g. when in jmx you have connections.value and connectionsdurationmean.value lookup of connections would match both, adding period at the end of grep fixes that)

Owner
tcurdt commented May 12, 2012

cksum does not seem to be widely available. Maybe use openssl instead to calc the md5?

Contributor
rstml commented May 12, 2012

Well, it's available on OSX, BSD, and Linux distros. From man cksum:

HISTORY
The cksum utility appeared in 4.4BSD.

openssl is less likely to be available by default, e.g. package may be not installed. Maybe we can just add check for availability of cksum command, if not available just ignore. There's also md5sum.

Owner
tcurdt commented May 12, 2012

ups! .. my fault must have been a typo when I was checking ... sorry for the noise

@tcurdt tcurdt merged commit 7352502 into tcurdt:master May 12, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment