On Sun, 22 Jan 2012, Simo Sorce wrote: > > > Shouldn't we just 'include' the original file and not copy it ? > > > If we include it a change in the file will be automatically picked up. > > We can't due to systemd design and use of it in 389-ds. > > > > LimitNOFILE needs to be changed in the service file directly, not in > > the environment file in /etc/sysconfig or otherwise it will not be > > picked up by the systemd. > > I am not talking about using /set/sysconfig options, but the .include > directive for service files. So, something like this for 389-ds? ----8<--------8<--------8<--------- [Unit] Description=389 Directory Server %i. BindTo=dirsrv.target After=dirsrv.target
[Service] Type=forking Environment=PIDDIR=/var/run/dirsrv EnvironmentFile=/etc/sysconfig/dirsrv EnvironmentFile=/etc/sysconfig/dirsrv-%i ExecStart=/usr/sbin/ns-slapd -D /etc/dirsrv/slapd-%i -i ${PIDDIR}/slapd-%i.pid -w ${PIDDIR}/slapd-%i.startpid .include /etc/sysconfig/dirsrv.systemd ----8<--------8<--------8<--------- and then with following /etc/sysconfig/dirsrv.systemd: ----8<--------8<--------8<--------- [Service] LimitNOFILE=8192 ----8<--------8<--------8<--------- local overrides can be created? It seems to work. You can't use %i macro in .include unfortunately so any configuration change will be global, not per service instance. # systemctl show dirsrv@IPA-LOCAL.service|grep LimitNOFILE LimitNOFILE=8192 # systemctl status dirsrv@IPA-LOCAL.service dirsrv@IPA-LOCAL.service - 389 Directory Server IPA-LOCAL. Loaded: loaded (/lib/systemd/system/dirsrv@.service; enabled) Active: active (running) since Mon, 23 Jan 2012 00:47:32 -0500; 11min ago Process: 30927 ExecStart=/usr/sbin/ns-slapd -D /etc/dirsrv/slapd-%i -i ${PIDDIR}/slapd-%i.pid -w ${PIDDIR}/slapd-%i.startpid (code=exited, status=0/SUCCESS) Main PID: 30929 (ns-slapd) CGroup: name=systemd:/system/dirsrv@.service/IPA-LOCAL └ 30929 /usr/sbin/ns-slapd -D /etc/dirsrv/slapd-IPA-LOCAL -i /var/run/dirsrv/slapd-IPA-LOCAL.p... I would still need a ticket to make appropriate removals in ipapython/platform/fedora16.py so that we would modify /etc/sysconfig/dirsrv.systemd and still do symlink re-linking in /etc/systemd/system/dirsrv.target.wants/. -- / Alexander Bokovoy _______________________________________________ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel