Skip to content

Add health check of database to kill bbs if the db is nonfunctional#134

Merged
geofffranks merged 3 commits intomainfrom
bbs-health-check
Dec 17, 2025
Merged

Add health check of database to kill bbs if the db is nonfunctional#134
geofffranks merged 3 commits intomainfrom
bbs-health-check

Conversation

@geofffranks
Copy link
Copy Markdown
Contributor

Summary

Adds a DB health check that will forcibly restart BBS if it is unable to perform a simple DB insert + retrieve within a configured timeout for a specified number of failures in a row, on a specified interval.

Defaults: 5s timeout, 3 attempts, 10s interval, healthcheck disabled.

Backward Compatibility

Breaking Change? No

ebroberson
ebroberson previously approved these changes Dec 11, 2025
Copy link
Copy Markdown
Contributor

@ebroberson ebroberson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-project-automation github-project-automation bot moved this from Inbox to Pending Merge | Prioritized in Application Runtime Platform Working Group Dec 11, 2025
Copy link
Copy Markdown
Member

@ameowlia ameowlia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are inconsistent with "healthcheck" and "health check" in variable names and logs. Can you pick your fave please?

@geofffranks
Copy link
Copy Markdown
Contributor Author

Ok, I think i addressed all the feedback. Thanks for catching those issues!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants