luci-app-statistics: only graph data supported by APC UPS 1707/head
authorguidosarducci <guidosarducci@users.noreply.github.com>
Sat, 3 Mar 2018 21:45:23 +0000 (13:45 -0800)
committerguidosarducci <guidosarducci@users.noreply.github.com>
Fri, 30 Mar 2018 03:44:22 +0000 (20:44 -0700)
commit387b89e6f90499a0ffe47ea926de789f83210c28
treec91b001ac554c2ee3cce8543148b9d5997862721
parent176f5fb94cd06acaa62f2f3174476f421a430fd8
luci-app-statistics: only graph data supported by APC UPS

Some graph definitions rely on data not supported across all APC UPSes.
Due to recent upstream changes in collectd, the daemon no longer creates
a NaN-filled .rrd file corresponding to any missing UPS data. Depending
on the connected UPS, this may result in some "broken" graphs on the Luci
Statistics page since rrdtool cannot find the expected .rrd file.

Include the add_supported() function to determine the UPS data available
at runtime and update any definitions of graphs to include only supported
data. For example, the whole chart stack of AC input and output voltages
will normally be "broken" if the UPS only measures AC input voltage. With
these changes, the output voltage graph definition is stripped out,
allowing the chart to render.

Make consistent use of data types and instances in graph definitions. All
definitions now use the same format with the 'instances' key. Unnecessary
'types' and 'sources' keys are removed.

Fix the definition of 'line frequency' graph, based on upstream collectd
apcups plugin code: type is 'frequency' but instance should be 'input'.

This also includes some code and whitespace cleanup.

Signed-off-by: Tony Ambardar <itugrok@yahoo.com>
(cherry picked from commit fa1f4c42a6e73483d8ed6284b177e5648f44a114)
applications/luci-app-statistics/luasrc/statistics/rrdtool/definitions/apcups.lua