On Sun, 11 Jan 2015 10:24:39 -0500 Steve Dickson <SteveD@xxxxxxxxxx> wrote: > Hey Neil, > > You being the architect of the systemd scribes ;-) I have a > couple questions. > > The nfs-server service brings both the rpc.mountd and rpc.idmapd > daemons up when the service is started, but only > brings rpc.mountd down when the service is stopped. > > I'm assuming that was done because the client was > using rpc.idmapd to do its id mapping, but that is no > longer the case with some clients. They use the > key rings via nfsidmap command to do the id mapping Your assumption is correct. Having nfs-client only require nfs-idmap on some configs is not easy to do with systemd... at least I don't know an easy way. We probably need to distribute two versions of some config file(s), one where the dependency exists, one where it doesn't. How does one tell whether idmapd is needed or not? > > So I'm thinking the nfs-server service should bring > both daemons up and down when the server is started > and stopped since the server is the only service using > the rpc.idmapd. > > My attempted at doing this is to change the > nfs-idmap service to do the following: > > [Unit] > Description=NFSv4 ID-name mapping service > Requires=var-lib-nfs-rpc_pipefs.mount > After=var-lib-nfs-rpc_pipefs.mount > After=network.target > PartOf=nfs-server.service > PartOf=nfs-utils.service > > Wants=nfs-config.service > After=nfs-config.service > > [Service] > EnvironmentFile=-/run/sysconfig/nfs-utils > Type=forking > ExecStart=/usr/sbin/rpc.idmapd $RPCIDMAPDARGS > > Almost exactly what the rpc-mountd service does. > > Now I thought the "PartOf=nfs-server.service" would cause > rpc.idmapd to come down when the server came down, but that > does not seem to be the case... The only way I can get > nfs-idmap service to come down is to explicitly stop it... > What am I missing? You are missing the same thing that I am missing. I wonder if systemd gets confused by multiple PartOf directives. The man page says: When systemd stops or restarts the units listed here, the action is propagated to this unit. So if systemd is told to stop nfs-server.service, it should also stop nfs-idmap.service... Maybe try with only one PartOf? Or with PartOf=nfs-server.service nfs-utils.service > > Secondly, in all of the services where rpcbind is needed you > reference the rpcbind.target instead of the rpcbind.service. > Why is that? Because rpcbind.target is a public name (even mentioned in doco). I had this idea that when systemd unit files from one package need to interact with those from another package, they should be careful to only use "public" interfaces in case one package changes. And I thought "rpcbind.target" was like a public interface. I no longer think that. The ".target" concept doesn't seem to be nearly as useful as I thought it would be. I would not object to changing nfs-utils to use "rpcbind.service" if that would help anyone at all. > > The reason I'm ask is, I'm seeing a problem where rpc.statd fails > to start because nfs_svc_create() fails. Meaning it was unable to > either create UPD/TCP sockets or the registrations to rpcbind fails. > > I'm thinking its the later due a race between rpcbind and statd > starting. I've seen races like before in with systemd services... but but but, the whole point of before/after is to avoid races... > > So I'm thinking that race would not exist if the rpc-statd service > would use rpcbind.service in the "Requires=" and "After=" > statements instead of rpcbind.target. Right?? Don't know. Try it and see. If it works, use it. Thanks, NeilBrown > > tia, > > steved. > -- > To unsubscribe from this list: send the line "unsubscribe linux-nfs" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html
Attachment:
pgpdiqDavVgxr.pgp
Description: OpenPGP digital signature
