Skip to content
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

Open
kodjo-anipah opened this issue Mar 22, 2024 · 5 comments

Comments

@kodjo-anipah
Copy link

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:

  1. Ask the user if a restart should happen
  2. Possibility to configure some of them as no-auto-restart

Why?

This was already suggested in the past
& and recently also reported as a bug in gz#220

Your Environment

  • Graylog Version: 5.2.4
  • OpenSearch Version:
  • MongoDB Version:
  • Operating System:
  • Browser version:
@tellistone
Copy link

Perhaps we might add a check for if the sidecar is running.

---> if it is running, restart it
---> if it is not running, do not start 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?

@kodjo-anipah
Copy link
Author

@tellistone yes sidecar is core

@tellistone
Copy link

can this be solved Graylog side, or will a new Sidecar build be needed?

@kodjo-anipah
Copy link
Author

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

@tellistone
Copy link

Yes please

@kodjo-anipah kodjo-anipah transferred this issue from Graylog2/graylog2-server Mar 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants