[rrd-developers] [patch] rrdcached init script and spec file

[Bernard Li]

Hi Daniel:

The init script does not work on my system (CentOS 4.x) as is, because
the `daemon` function which I have does not support --pidfile -- is
that argument necessary?

Also, as discussed previously, I think it would be a good idea to
create a 'rrdcached' user and group and start the daemon as that user
instead of nobody.  For application-specific (eg. Ganglia)
implementations, we can just put the necessary users (such as nobody,
apache, ganglia) in the rrdcached group.

Thanks,

Bernard

On Tue, Oct 7, 2008 at 3:58 PM, Bernard Li <bernard at vanhpc.org> wrote:
> Hi all:
>
> I have some comments regarding the rrdtool spec file that is in trunk
> now (which includes changes to incorporate rrdcached).
>
> First of all, thanks to Daniel for putting this together (saves me the
> work, heh).
>
> However, I have two comments:
>
> 1) I think we should break this out as a separate subpackage such as
> rrdtool-rrdcached as I don't think rrdcached is needed by your
> everyday installation (only large installations).  Incorporating it in
> the main rrdtool package and especially by including an init script
> gives users the impression that this is something that is needed by
> everybody, which I don't think is the case.
>
> 2) By default, the rrdcached daemon is started once you install the
> RPM.  While I think it is fine to add rrdcached as a service, I don't
> think it's a good idea to start up the daemon by default especially
> when one might want to make some configuration changes prior starting.
>  It should be left to the user to start/top the daemon as they like.
>
> If you guys agree, I can go ahead and create a patch for the above two
> points.  I may have additional comments after I've had some time to
> play with the new code.
>
> Thanks,
>
> Bernard
>