Describe the bug
config-loader sources /etc/docker-salt/runtime/functions.sh which calls set -o errexit which will cause the script to exit on any command failure.

It then later runs killall salt-master; killall -9 salt-master

If these commands return non-zero exit (failing to kill processes), the whole script exits and the subsequent daemon restart commands are never called, leaving the daemons stopped.

Looking at the process tree it appears that the salt-master processes are actually of the form:

/opt/saltstack/salt/bin/python3.10 /usr/bin/salt-master

This means that killall will always fail as it matches process, not arguments.

To match arguments, avoid premature exit, and also catch any runaway api processes, this might be a good alternative:

pkill -f "salt-(master|api)" || true
pkill -f -9 "salt-(master|api)" || true

Happy to generate a PR if this solution suits, or you have a different one in mind?