Gday folks.

I'm fiddling around with remote RRD storage via rrdcached.    Most of the rrdtool stuff is wrapped in includes/rrdtool.inc.php which is wonderful from this perspective.

Not within this include is the usage of is_file() to determine the existence of an rrd file.

In discovery the pattern is something like

 If( ! is_file( some.rrd ) ) { rrdtool_create ( … )}

In graphing the pattern is something like

 If ( is_file( some.rrd ) { graph some stuff… }

Would there be any support for moving this usage into a rrdtool_file_exists( ) thin wrapper for is_file ?

This would make it much easier to extended includes/rrdtool.inc.php to support storage of RRD's on remote storage (and potentially distribute over multiple remote storage devices ..)

Thoughts?

I had a quick check for usage of scandir/readdir to determine all the rrd's in a 'device' directory, but this doesn't seem to be a common pattern.

My other evil thought is to use rename_function/override_function to work around this, but it seems very very WRONG.

Cheers,

  Peter