luci-app-statistics: Remove usually empty std.dev. graph from ping page 638/head
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 12:34:10 +0000 (14:34 +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>
applications/luci-app-statistics/luasrc/statistics/rrdtool/definitions/ping.lua

index f0a5948..6ecdb5c 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