Bug#768168: watchdog does not start at boot

[Michael Biebl]

Am 12.11.2014 um 14:29 schrieb Uwe Storbeck:
> On Nov 11, Michael Biebl wrote:
>> Please boot with systemd.log_level=debug on the kernel command line, the
>> attach the output of "journalctl -alb"
> 
> I guess these are the relevant lines:
> 
>   Found ordering cycle on graphical.target/start
>   Found dependency on multi-user.target/start
>   Found dependency on watchdog.service/start
>   Found dependency on graphical.target/start
>   Breaking ordering cycle by deleting job watchdog.service/start
> 
> The part of the journal after hardware initialization is attached.


Yeah, it shows that watchdog.service is involved in dependency cycle and
systemd therefore removes it from the transaction.

Here's the service file:

> [Unit]
> Description=watchdog daemon
> After=systemd-update-utmp.service

Since the service uses DefaultDependencies=yes (the default), this
explicit after is not necessary.

> After=runlevel1.target runlevel2.target runlevel3.target runlevel4.target runlevel5.target

I don't think this does and please don't use that in a service file. We
don't really want to drag that legacy concept of SysV runlevels into
native .service files.
	
> 
> [Service]
> Type=forking
> EnvironmentFile=/etc/default/watchdog
> ExecStartPre=/bin/sh -c '[ -z "${watchdog_module}" ] || [ "${watchdog_module}" = "none" ] || echo /sbin/modprobe $watchdog_module'

Couldn't you just run "ExecStartPre=-/sbin/modprobe $watchdog_module"?
The "-" would ignore any errors

> ExecStartPre=/bin/systemctl stop wd_keepalive

Calling systemctl from service files is highly discouraged, I'd go as
far as say forbidden unless you know exactly what you're doing.
Can you elaborate why this is necessary?

> ExecStart=/usr/sbin/watchdog $watchdog_options
> ExecStopPost=/bin/systemctl start wd_keepalive

Same here. Does that mean, on shutdown, when all services are about to
be stopped, you start wd_keepalive? Why?

> [Install]
> WantedBy=multi-user.target



Michael
-- 
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://alioth-lists.debian.net/pipermail/pkg-systemd-maintainers/attachments/20141112/bfff020b/attachment-0002.sig>