cmd/ctrld: remove firerouter_dns dependency in systemd unit on firewalla

On firewalla, firerouter_dns is a shell script, which forks dnsmasq
processes. At the end of ctrld stopping process, ctrld attempts to
restart firerouter_dns. The systemd v237 on firewalla somehow hangs,
because ctrld depends on firerouter_dns, but attempts to restart it
before ctrld stopping.

However, thing in firewalla is ephemeral, so after reboot, ctrld is
re-installed at the end of boot process. Thus, ctrld don't have to
depend on any services.
This commit is contained in:
Cuong Manh Le
2023-06-23 10:32:19 +07:00
committed by Cuong Manh Le
parent 2f46d512c6
commit 3f211d3cc2

View File

@@ -33,13 +33,6 @@ func setDependencies(svc *service.Config) {
svc.Dependencies = append(svc.Dependencies, "Wants=dnsmasq.service")
svc.Dependencies = append(svc.Dependencies, "After=dnsmasq.service")
}
// On Firewalla, ctrld needs to start after firerouter_{dhcp,dns}, so it can read leases file.
if router.Name() == router.Firewalla {
svc.Dependencies = append(svc.Dependencies, "Wants=firerouter_dhcp.service")
svc.Dependencies = append(svc.Dependencies, "After=firerouter_dhcp.service")
svc.Dependencies = append(svc.Dependencies, "Wants=firerouter_dns.service")
svc.Dependencies = append(svc.Dependencies, "After=firerouter_dns.service")
}
}
func setWorkingDirectory(svc *service.Config, dir string) {