Force reload in frankenphp service configuration

[kitro]

@henderkes This resolve the symlink issue for the symlink deployment strategy.
alpine already have the force flag.

[henderkes]

I'm not sure if this should be made the default. I've tried to mirror Caddy's service definitions as closely as possible.

[henderkes]

@dunglas @alexandre-daubois thoughts on this?

@kitro you can use "frankenphp reload" instead of systemctl reload frankenphp, by the way. The command really only sends a http request to the admin API, so it doesn't have to happen initiated by systemd.

[kitro]

@henderkes Yes, It can be used the frankenphp reload -c /etc/frankenphp/Caddyfile --force.
As you see the issue of symlink It's been happening a lot lately, as many people using the symbolic link deployment and the service manager to reload.

From frankenphp perspective used as drop-in-replacement of fpm I think is good to make it default.

[alexandre-daubois]

This feels weird because --force wasn't introduce for this purpose (see caddyserver/dist#84).

Maybe we should look into Caddy's resolve_root_symlink option?

[henderkes]

This feels weird because --force wasn't introduce for this purpose (see caddyserver/dist#84).

Maybe we should look into Caddy's resolve_root_symlink option?

There's nothing to really change there. Realpath prevents non-worker script changes from being picked up immediately and workers need to be restarted anyway.

[alexandre-daubois]

Well maybe it makes sense to add --force after all. We already have it for Alpine, so it would sync everything on this side.

[kitro]

I think is good to merge.
Thanks!

[henderkes]

Okay, no objections from my side then.