RE: [RFC PATCH v4] ndctl: monitor: add ndctl monitor daemon

> > I prefer to add an EnvironmentFile like /etc/sysconfig/ndctl/monitor > to systemd rather than add a configuration file. According to [1], > environment variable substitution is supported in systemd.service, so > we can define the variables through "EnvironmentFile=/etc/sysconfig/ndctl/monitor". > In this fashion, we do not need to add any extra codes to parse the configuration file. > > In this case, [--conf-file] option is not necessary either. > According to [2], sytemd units can be instantiated from a template > file, thus we only need to add a template unit file in advance. > If user wants to run multiple monitors with different configurations, > they can differentiate them by adding multiple EnvironmentFiles, like /etc/sysconfig/ndctl/<monitor1...n>. > Then the monitors can be started by command like "# systemctl start > ndctl-monitor@<monitor1...n>.service". > > When the monitors started by hand, it will do not need any > configuration files, because we can add options and parameters directly. > > [1]https://www.freedesktop.org/software/systemd/man/systemd.service.ht > ml > [2]https://www.freedesktop.org/software/systemd/man/systemd.unit.html This seems to needlessly tie ndctl to systemd, it should be able to operate without requiring systemd. I expect it would be straightforward to copy the configuration file implementation from git.