systemd: drop WantedBy directive from kresd.service

Since kresd is socket-activated, there is no need for the WantedBy directive in the .service file itself. As long as the .socket files are installed, the service will run when it is used.

mentioned in commit ed62cc88

Please note that currently we intent to switch off socket activation by default, and to support manual service management. To be clear: We are not removing socket activation, but do not want to force people to use it. Users will have choice.

assigned to @tkrizek

@tkrizek Please decide what to do with this MR and merge/close it as necessary.

It seems to me the WantedBy=sockets.target doesn't even belong in the service file at all. From man systemd.special:

Services that can be socket-activated shall add Wants= dependencies to this
unit for their socket unit during installation. This is best configured via a
WantedBy=sockets.target in the socket unit's "[Install]" section.

I think we should delete it as proposed and keep it only in the socket's unit file.

@dkg Could you please update the commit message to make it clear this directive was pretty much useless in the service file?

added 3 commits

• e217ec1f...87a1805c - 2 commits from branch knot:master
• 2476e5ff - systemd: drop WantedBy directive from kresd.service

Compare with previous version

I've updated the commit message and rebased it against master.

merged

mentioned in commit 20f033ca

Thanks!

added bug label

Fixed in master, deleting branch.