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 <Brian.Scott at bunyatech.com.au> 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
Xymon mailing list Xymon at xymon.com http://lists.xymon.com/mailman/listinfo/xymon