| Commit message (Collapse) | Author | Files | Lines |
|
|
|
The IPMI collectd plugin uses parentheses in filenames (`fanspeed-FAN 1 fan_cooling (29.1).rrd`), so we need to escape them before passing them to rrdtool.
|
|
From the docs:
"No warning is generated if the variable does not exist. That means empty() is
essentially the concise equivalent to !isset($var) || $var == false."
This reverts commit f2a4593558b3a3ec696e9cc1de93fa898b9521bb.
Reported-By: Benoit Monin (https://github.com/pommi/CGP/commit/f2a4593)
|
|
|
|
Reported-By: Julien Genestoux <julien.genestoux@gmail.com>
(https://github.com/pommi/CGP/issues/24)
|
|
Fix some potential undefined index errors by using the empty function to
access ds_names in type other than default.
|
|
Fix php notices emitted when generating graphs of ping, sensors and
hddtemp plugins.
|
|
|
|
Since commit v0.3-97-g5793a8c [integrate jsrrdgraph in CGP], double quotes have
been removed around filenames and legend because it shouldn't be necessary (and
jsrrdgraph didn't support it).
This broke graphs rendered using jsrrdgraph with files including spaces in it's
name. Hopefully this is fixed now by rawurlencoding them (but the /).
|
|
Since commit v0.3-97-g5793a8c [integrate jsrrdgraph in CGP], double quotes have
been removed around filenames and legend because it shouldn't be necessary (and
jsrrdgraph didn't support it).
This broke graphs rendered by rrdtool with files including a colon in it's
name. Hopefully this is fixed now by double escaping them.
Reported-by: Poil <poil@quake.fr> (https://github.com/pommi/CGP/issues/20)
|
|
It caused an "Unknown argument: undefined" error when using canvas.
|
|
Use $_SERVER['PATH_INFO'] instead.
|
|
|
|
|
|
|
|
|
|
|
|
Reference: http://rrdtool.mirror.graemef.net/doc/rrdfetch.en.html#IAT_STYLE_TIME_SPECIFICATION
|
|
Reported-by: Philipp Hellmich <Philipp.Hellmich@Bertelsmann.de>
|
|
Reported-by: Philipp Hellmich <Philipp.Hellmich@Bertelsmann.de>
|
|
This problem showed up with IPv6 addresses in the NTP plugin.
Forgotten in commit:
v0.3-51-g330c94f [type: escape colons in filenames used in rrd commandline]
|
|
|
|
this should keep the error.log clean
|
|
|
|
Since commit collectd-4.10.0-88-g02e12db the varnish plugin groups collected
values in categories. The collectd filestructure used for varnish is now:
<plugin>-<category>-<plugin_instance>/<type>-<type_instance>
Because this isn't distinguishable from a regular plugin like df, ...
df-var-tmp/df_complex-free.rrd ("var" isn't the category here)
... the category is only set with the varnish plugin.
Reported-by: Jonathan Huot <jonathan.huot@gmail.com>
|
|
|
|
This is a correction to commit:
v0.3-56-g69bea69 [only use raw totals when no scale is set]
|
|
When scale is set, the total will be set twice (also in part below), which will
result in this error: "ERROR: Attempting to reuse 'tot_98574167'"
Bug introduced in commit:
v0.3-25-g19ed46a [type/GenericIO: add total calculations in IO graphs]
|
|
A colon ':' in a legend argument will mark the end of the legend:
http://oss.oetiker.ch/rrdtool/doc/rrdgraph_graph.en.html#INOTES_on_legend_arguments
Reported-by: Benjamin Dupuis <poil@quake.fr>
|
|
Use 'value' instead.
|
|
Including a lot of undefined $_GET values.
|
|
|
|
|
|
When no colors are defined they will be generated by function rainbow_colors.
|
|
Function fill_ds_names is renamed to parse_ds_names and now also
appends spaces to ds_names to make them equal of lenght.
|
|
instance
|
|
|
|
Like mentioned in the comment on the website below, I think there is no
problem a source contains dots. For now, this line is removed.
http://pommi.nethuis.nl/2010/01/collectd-graph-panel-v0-2/#comment-177
|
|
Since I'm using a non standard installation of rrdtool, I always
have to define the --font options when colling the rrdtool binary.
I tried to implement this change in a general way to define locally
rrdtool options in the config.local.php.
Signed-off-by: Edmondo Tommasina <edmondo@eriadon.com>
|
|
Since commit 95c70c919 [rewrite of type classes] cpu graphs of hosts running
a 2.4 Linux kernel or Solaris were not displayed. Function rrd_get_sources
must return an array with a consecutive index of numbers.
|
|
The identifier is needed to send the FLUSH command to the UnixSock plugin.
Partly revert of commit:
596dd7d41 [make rrdfiles using utf8 and spaces work]
|
|
|
|
|
|
|
|
|
|
Types GenenericIO and GenericStacked were not included in commit:
596dd7d414 [make rrdfiles using utf8 and spaces work]
|
|
|
|
|
|
|
|
A constructor is added to the Type_Default class. The constructor will
parse GET values (such as host, plugin, pinstance, type, tinstance,
seconds), create an array of all needed rrd files to generate a graph and
substract identifiers from these rrd files.
Because of the constructor (and related functions) it is not needed to
define an array of tinstances to be grouped and shown in one graph. Also
$obj->args don't have to be defined per plugin. This will result in
smaller plugin files.
The type classes are based on the fact that a plugin has multiple type
instances OR multiple rrd data sources. This is called the source and is
retrieved by rrd_get_sources in each rrd_gen_graph function. Also
variables in function rrd_gen_graph have been renamed to better ones.
|