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 <[EMAIL PROTECTED]> 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 > _______________________________________________ rrd-developers mailing list rrd-developers@lists.oetiker.ch https://lists.oetiker.ch/cgi-bin/listinfo/rrd-developers