-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Optional watchdog for "adapter disconnected"-type events (non-node-crash) #23043
Conversation
Looks good! I think the best place to document this is above here |
That would only provide the information on Linux side though. Do you want to duplicate the paragraph on all OS pages? |
I think for each setup the way to enable this is a bit different, but Im not sure if we want it for the Docker setup for example. |
For containerized setups, I guess it can remove some overhead, by avoiding container reset, and just trying to launch the controller again. |
Im thinking that it might be easier to enable this through an env var, |
|
Yes, and with everything that are not csv numbers, z2m should refuse to start |
For the docs, Im wondering if a new Watchdog page under Configuration makes sense |
How about under |
Thats also fine |
Add basic, optional, watchdog to handle "soft failures" for bare Z2M users instead of requiring the use of a process watchdog. Should handle any Z2M failure that would result in the node process being stopped (like "adapter disconnected" event).
npm run start
=> no watchdogZ2M_WATCHDOG=default npm run start
=> watchdog with default delays (1min, 5min, 15min, 30min, 60min)Z2M_WATCHDOG=minutes_csv npm run start
=> watchdog with custom delays (example:Z2M_WATCHDOG=0.5,3,6,15 npm run start
)Z2M_WATCHDOG
acceptsdefault
as value or a list of comma-separated integers/floats. Other values/formats (invalid) will prevent Z2M from starting.Note: This does not handle node crashes, that's better suited for a process watchdog (and already widely available).