Opened 3 years ago

Last modified 2 months ago

#1158 new enhancement

make SNMP multigraphs look like MRTG graphs

Reported by: anarcat Owned by: nobody
Priority: normal Milestone: Munin 2.0.0
Component: plugins Version: 1.4.6
Severity: normal Keywords: mrtg pola patch


Here at Koumbit we have done a patch to make the SNMP multigraphs look like MRTG graphs. There are multiple reasons for this:

  1. POLA (Principle of Least Astonishment) - most graphing software does it that way and it's confusing to switch to Munin and try to figure it out from there
  2. Comparing graphs is easier - with the current graphs, comparing the graph from both ends of a cross-connect in switches si very hard as the graphs are reversed, and simply can't be compared without being either very good with visuals or using graphical software.

I have attached graph examples of two interfaces that are supposed to match, to give an idea of the latter problem.

I have also attached a patch which makes the received graph be a green AREA and sent graph be a blue LINE1, both on the positive side. I think it covers most of the transition.

I know this is a sensitive and highly debatable issue. I am also aware there is a workaround proposed in the FAQ. However this workaround assumes that I create a snippet for every single port for every switch we monitor, which currently means around 300 code snippets - basically defeating the whole idea behind the SNMP multigraph plugin.

I would go as far as saying that the default if_ graphs should also be changed so they can be compared with the SNMP graphs, but I am not sure I can win that fight, so I write provide a patch only if there is some sympathy with it. ;)

Attachments (5)

if_50-day-before.png (24.2 KB) - added by anarcat 3 years ago.
interface 50, before change
if_1-day-before.png (23.9 KB) - added by anarcat 3 years ago.
interface 1, before change
if_50-day-after.png (20.9 KB) - added by anarcat 3 years ago.
if_1-day-after.png (20.5 KB) - added by anarcat 3 years ago.
munin_mrtg_graphs.patch (4.0 KB) - added by anarcat 3 years ago.

Download all attachments as: .zip

Change History (6)

Changed 3 years ago by anarcat

interface 50, before change

Changed 3 years ago by anarcat

interface 1, before change

Changed 3 years ago by anarcat

Changed 3 years ago by anarcat

Changed 3 years ago by anarcat

comment:1 Changed 2 years ago by kenyon

  • Keywords patch added

You make a good argument for the MRTG style, but there are so many plugins, both in the main distribution and in the contrib repository, that use the negative-positive style. In addition to if_, I know of netstat_multi, ip_, diskstats, swap, and tor_bandwidth_usage. It seems that some of these would not be very readable in MRTG style. Personally, I find it pretty easy to see that the switch cross-connect interface graphs match in shape and in numbers, with the current munin style.

Maybe you should post a message to the munin-users mailing list to get opinions on this. Otherwise, I'm afraid we might have to close this without applying the patch, because the munin negative-positive style seems pretty well entrenched into so many munin plugins.

Note: See TracTickets for help on using tickets.