On Wed, 19 Jul 2023 11:26:44 +0200 =?utf-8?q?Lars_Dr=C3=B6ge?=
<[email protected]> wrote:
> Package: sanlock
> Severity: important
> Justification: renders package unusable
>
> Dear Maintainer,
>
> thank you for maintaining the sanlock package. I may have found a version
> mismatch in the included wdmd.service file and a lib.
>
> * What led up to the situation?
> Try to start the wdmd.service, as it was not succesfully pulled in by
> sanlock.service.
> * What exactly did you do (or not do) that was effective (or
> ineffective)?
> systemctl start wdmd.service
> * What was the outcome of this action?
> Error message: Usage: /etc/init.d/wdmd {start|stop|status|restart}
> * What outcome did you expect instead?
> wdmd.service starts
>
> The wdmd.service contains this line in the [Service] section:
>
> ExecStartPre=/lib/systemd/systemd-wdmd watchdog-check
>
> But /lib/systemd/systemd-wdmd does not support the argument "watchdog-check",
> and throws an error instead.
> All related files are part of the sanlock package.
>
It seems that we use a rather old version of this file, created by the
first maintainer of the package. Upstream has a different file that has
the "watchdog-check" functionality implemented.
> I have found this behavior in
> - sanlock 3.8.2-2 in Debian Bullseye
> - sanlock 3.8.5-1+b1 in Debian Bookworm
>
> Is it safe to remove the ExecStartPre= line and skip the check?
>
As a temporary solution it most likely is, the check can be run manually
if you want it. The "watchdog-check" only does two things, and is easy
to replicate.
Ensuring that the wdmd.service is stopped, then executing
wdmd --probe
If it is successful, everything is well and nothing more needs to be
done, if it fails, you should try to load the "softdog" kernel module
with
modprobe softdog && udevadm settle
The relevant part, that is missing from the Debian package, can be found
here [1].
For unstable, I will try to use upstream files for both the systemd
service's as well as their init.d scripts and configuration files. For
bookworm the easiest solution might be to just copy in the two missing
functions.
Regards,
Håvard
[1] https://sources.debian.org/src/sanlock/3.8.5-1/init.d/wdmd/#L34-L56
