-
Notifications
You must be signed in to change notification settings - Fork 312
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 automated health checks #733
Add automated health checks #733
Conversation
Signed-off-by: Dylan Schultz <hello@schultzie.dev>
Signed-off-by: Dylan Schultz <hello@schultzie.dev>
Signed-off-by: Dylan Schultz <hello@schultzie.dev>
This is excellent, thanks so much @dylanschultzie! The only part I'm considering changing is whether we should make that I'll give this a bit of a test ASAP |
@tombeynon the timeout param is set to every 24 hours, and grace period to 12. I tried to find a reasonable medium so as to not dilute the networks.local.json file, BUT I'm more than happy to add in extra params if you'd prefer. |
Excuse me, it's the other way around - the grace period is 24 hours, and timeout is 12 hours. |
Signed-off-by: Dylan Schultz <hello@schultzie.dev>
Signed-off-by: Dylan Schultz <hello@schultzie.dev>
This is awesome - can't wait to give it a go! |
All working great, thanks so much for this and the extra config additions @dylanschultzie 🚀 |
There are really 3 parts to this PR:
apiKey
property is added, and is created based off of thename
property (which defaults tonetworkName
)