Brian
Thanks for your work on this. Hopefully your patches will make it to
mainline source.
A question about the patch for graphs.cfg. The RRD files I have named
ifmib.<ifname>.rrd all have InOctets and InUcastPkts. I think these are
created by Devmon. I would think that most deployments using SNMP with
Xymon use devmon for this, so there may be a lot of RRD files using the
pre-patched version of the [ifmib] section from graphs.cfg, and their
presentation would break if your patch was applied. I wonder if it's a
better idea to patch snmpmibs.cfg to use inOctets instead of ifInOctets,
etc, and maintain compatibility with devmon.
Cheers
Jeremy
On Thu, 1 Jul 2021 at 15:10, Brian Scott <user-7c0060721e1b@xymon.invalid>
wrote:
Hi,
I've been poking about with SNMP data collection with xymon for the last
few days and have a few patches that may be useful for the next release,
or for others exploring SNMP processing.
The one for snmpmibs.cfg is from a post by Jeremy Laidman a while back
so may already be in source control. It just removes a unwanted '(' from
a mib definition. Thanks.
The graphs.cfg patch is for typing mistake in the graph definition for
ifmib and ifmib1.
The xymon-snmpcollect.c one is for a programming error in automatically
finding the snmphosts.cfg file if not specified on the command line.
Finally, the showgraph.c patch is an enhancement to create a new
substitutable, @BLANK@, that outputs blanks to take up the same space as
@RRDPARAM@ in the graph legend.
I've done a bit of a write-up of what I've done/found which I'll send a
link to in the next few days.
Cheers,
Brian Scott