feat: Docker health check for bot container readiness#8
Open
interplanetarychris wants to merge 2 commits intojgarzik:mainfrom
Open
feat: Docker health check for bot container readiness#8interplanetarychris wants to merge 2 commits intojgarzik:mainfrom
interplanetarychris wants to merge 2 commits intojgarzik:mainfrom
Conversation
Add HEALTHCHECK to bot containers that polls the OpenClaw /health endpoint. Dashboard now uses actual container health status instead of time-based heuristic to determine when bot is ready. - Add Healthcheck config when creating containers (2s interval, 30 retries) - Add health field to ContainerStatus type (none/starting/healthy/unhealthy) - Update getContainerStatus to extract health from Docker inspect - Update BotCard to use health status for starting/running distinction Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Containers created before the healthcheck feature was added have health='none'. These should be treated as running, not starting. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add Docker HEALTHCHECK to bot containers so the dashboard can show accurate startup status instead of using a time-based heuristic.
Problem
Previously, the dashboard used an 8-second timer to guess when a bot was ready. This was unreliable - OpenClaw can take 10+ seconds to start, leading to "running" status while the bot was still initializing.
Solution
/healthendpoint)starting→healthy) instead of time heuristicChanges
src/services/DockerService.tssrc/types/container.tsHealthStatustype andhealthfielddashboard/src/types.tsHealthStatustype andhealthfielddashboard/src/dashboard/BotCard.tsxHealth check config
Test plan
/healthresponds🤖 Generated with Claude Code