-
Notifications
You must be signed in to change notification settings - Fork 56
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
Sidecar should not always automatically restart on configuration change #494
Comments
Perhaps we might add a check for if the sidecar is running. ---> if it is running, restart it seems like a reasonable compromise - the purpose of the restart is to apply the config to a running sidecar. No need to stop-start it, if it is already stopped - it'll pick up the new config when the user next starts it. Should this be in core? |
@tellistone yes sidecar is core |
can this be solved Graylog side, or will a new Sidecar build be needed? |
Just by having a glance at the code, I think the sidecar would be the place to fix it as this is where the restart is actually happening, so that would need a new build. Should I move it over to the sidecar project? @tellistone |
Yes please |
What?
By design when we apply a new configuration to a Sidecar, that specific collector gets restarted.
This can cause problems for users who want to update the configuration without actually restarting the service.
Possible solutions would be:
Why?
This was already suggested in the past
& and recently also reported as a bug in gz#220
Your Environment
The text was updated successfully, but these errors were encountered: