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

Add option to configure Systemd Watchdog #644

Closed
MrSerth opened this issue Aug 10, 2024 · 1 comment · Fixed by #689
Closed

Add option to configure Systemd Watchdog #644

MrSerth opened this issue Aug 10, 2024 · 1 comment · Fixed by #689
Labels
enhancement New feature or request

Comments

@MrSerth
Copy link
Member

MrSerth commented Aug 10, 2024

Not all operating systems use systemd, and thus don't necessarily support the systemd watchdog. One example would be macOS 💻. However, with the current setup, an error is issues that the systemd watchdog is unsupported on each start of Poseidon.

When error reporting to Sentry is enabled, consequentially, each start of Poseidon on macOS reports a new event (which I have to delete manually). It would be great if we could change the current behavior.

I see the following potential approaches:

  1. Add a config option to disable the systemd watchdog
  2. Lower the level of the "Systemd Watchdog not supported" error (maybe similar to "Systemd Readiness Notification not supported")
  3. Couple systemd readiness notifications and systemd Watchdog: Return from notifySystemd, when systemd readiness notifications are not supported
  4. (Disable the watchdog for non-Linux systems)
@MrSerth MrSerth added the enhancement New feature or request label Aug 10, 2024
@mpass99
Copy link
Collaborator

mpass99 commented Sep 13, 2024

Thanks for already proposing approaches. In #689 I've implemented approach 2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants