[patch] Show correct values in net-snmp CPU Usage graph
Description
While working on a graph that I wanted to look like the CPU Usage graph, I noticed that the values displayed in the bottom of the net-snmp CPU Usage graph are wrong. Instead of displaying the actual values, the graph shows the aggregated value as used in the graph.
See attached screenshots and patch.
Acceptance / Success Criteria
None
Attachments
3
Lucidchart Diagrams
Activity
Show:
Ken Eshelby September 13, 2011 at 10:17 PM
I found a ton of those min/max defs all over defaults as well. It will be a while before I can submit a patch.
Seth Leger September 7, 2011 at 11:50 AM
I've committed this change to the 1.8 and 1.10 branches. Great catch Dominik! Thanks for the patch.
commit 2988fd504386e792194b0bd7334befea1a9405f8
Seth Leger August 31, 2011 at 8:34 PM
OK I understand now, the values at the bottom of the page are derived from the stacked graph value instead of the values for each real column. UGLY, this needs to be fixed ASAP. In fact, I think I can remember people complaining about these values a few months ago.
Seth Leger August 31, 2011 at 8:26 PM
This looks like a pretty severe bug... I can't understand why the graph definition was ever written like it was. After I get some other people to review this change, I'll try and apply the patch to the 1.10 branch.
Dominik March 10, 2011 at 7:55 AM
The patch also removes some - imho - unnecessary lines from that graph definition. It might be that this bug is present in more graphs, but I only looked at this one yet.
While working on a graph that I wanted to look like the CPU Usage graph, I noticed that the values displayed in the bottom of the net-snmp CPU Usage graph are wrong. Instead of displaying the actual values, the graph shows the aggregated value as used in the graph.
See attached screenshots and patch.