-
Notifications
You must be signed in to change notification settings - Fork 580
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Satellite reload fails with Systemd on config sync (git master only) #6127
Comments
This clearly is a problem how Systemd handles the reload. If I start icinga2 on the CLI, the reload works like a charm. Might be related to #6082 but in this specific case, the PID file is not removed.
|
Investigation from It started to work after a while of testing.
|
I need to investigate further on this, please add your feedback. |
Does this happen when running the safe-reload script? |
This is what Systemd invokes I believe ... I don't understand how the current reload was changed, afaik you're now using Systemd signals to tell Systemd to fire a reload, which in turn calls |
There are two changes in this:
But none of these changes affect the init scripts (except offering to use WatchdogSec=). |
Same issue as #6082 |
I would need custom RPM artifacts from that branch put into 2 VMs for proper tests, that's time I cannot afford in the coming weeks. I'll test #6163 once it is merged, sorry. |
Expected Behavior
Automated config reload takes place.
Current Behavior
Config reload takes place, but child process cannot take over as the socket is not entirely closed.
Possible Solution
Investigate on why Systemd thinks that the process failed with exit code 1. It seems that the FDs are not properly closed on reload, which causes the child process not to entirely take over.
Seems to come from the change with the reload in #5996
Steps to Reproduce (for bugs)
Terminal 2
Terminal 1
Context
Testing the latest development snapshot.
Your Environment
icinga2 --version
):icinga2 feature list
):icinga2 daemon -C
):zones.conf
file (oricinga2 object list --type Endpoint
andicinga2 object list --type Zone
) from all affected nodes.The text was updated successfully, but these errors were encountered: