luci-app-statistics: Remove usually empty std.dev. graph from ping page
authorHannu Nyman <hannu.nyman@iki.fi>
Tue, 2 Feb 2016 12:22:16 +0000 (14:22 +0200)
committerHannu Nyman <hannu.nyman@iki.fi>
Tue, 2 Feb 2016 13:12:33 +0000 (15:12 +0200)
Remove the usually empty standard deviation graph from the ping page.

The graph is empty for most users as collectd measures standard deviation
of individual pings inside the general interval of statistics collection.
Default setting for both ping interval and general collection interval
is 30s in Luci statistics, meaning just 1 ping per interval, which
leads to empty graph.

(To provide relveant data, the ping interval should be 1/4-1/5 of the
general collection interval. Even then the graph does not look very
informative due to different scaling than the latency graph.)

Note that this commit does not change collectd itself, which continues
to collect and calculate also the std.dev. data, which can be fetched
with 'rrdtool' if needed.

Signed-off-by: Hannu Nyman <hannu.nyman@iki.fi>
(cherry-picked from 8f8957f90f2c593e233d867cab48519a2215b8a0)

applications/luci-app-statistics/luasrc/statistics/rrdtool/definitions/ping.lua

index f0a59481695e797ef9e61a49db4dd5f1772acf9e..6ecdb5c8ad3f233255654fdef672e81baccb5c87 100644 (file)
@@ -23,16 +23,6 @@ function rrdargs( graph, plugin, plugin_instance, dtype )
                        types   = { "ping_droprate" },
                        options = { ping_droprate = {
                                noarea = true, overlay = true, title = "%di" } }
-               } },
-
-               -- Ping standard deviation
-               { title = "%H: ICMP Standard Deviation",
-                 vlabel = "ms",
-                 number_format = "%5.2lf ms",
-                 data = {
-                       types   = { "ping_stddev" },
-                       options = { ping_stddev = {
-                               noarea = true, overlay = true, title = "%di" } }
-               } },
+               } }
        }
 end