Chore: add Dockerfile HEALTHCHECK and debug-log silent catch blocks#11478
Merged
vincentkoc merged 5 commits intoopenclaw:mainfrom Mar 2, 2026
Merged
Chore: add Dockerfile HEALTHCHECK and debug-log silent catch blocks#11478vincentkoc merged 5 commits intoopenclaw:mainfrom
vincentkoc merged 5 commits intoopenclaw:mainfrom
Conversation
bfc1ccb to
f92900f
Compare
|
This pull request has been automatically marked as stale due to inactivity. |
Member
|
Rebased this PR branch onto current What changed:
Current diff is now a single, mergeable Dockerfile change. |
1 task
safzanpirani
pushed a commit
to safzanpirani/clawdbot
that referenced
this pull request
Mar 2, 2026
…penclaw#11478) * Docker: add /healthz-based container HEALTHCHECK * Docs/Docker: document built-in image HEALTHCHECK * Changelog: note Dockerfile healthcheck probe * Docs/Docker: explain HEALTHCHECK behavior in plain language * Docker: relax HEALTHCHECK interval to 3m --------- Co-authored-by: Vincent Koc <vincentkoc@ieee.org>
steipete
pushed a commit
to Sid-Qin/openclaw
that referenced
this pull request
Mar 2, 2026
…penclaw#11478) * Docker: add /healthz-based container HEALTHCHECK * Docs/Docker: document built-in image HEALTHCHECK * Changelog: note Dockerfile healthcheck probe * Docs/Docker: explain HEALTHCHECK behavior in plain language * Docker: relax HEALTHCHECK interval to 3m --------- Co-authored-by: Vincent Koc <vincentkoc@ieee.org>
hanqizheng
pushed a commit
to hanqizheng/openclaw
that referenced
this pull request
Mar 2, 2026
…penclaw#11478) * Docker: add /healthz-based container HEALTHCHECK * Docs/Docker: document built-in image HEALTHCHECK * Changelog: note Dockerfile healthcheck probe * Docs/Docker: explain HEALTHCHECK behavior in plain language * Docker: relax HEALTHCHECK interval to 3m --------- Co-authored-by: Vincent Koc <vincentkoc@ieee.org>
execute008
pushed a commit
to execute008/openclaw
that referenced
this pull request
Mar 2, 2026
…penclaw#11478) * Docker: add /healthz-based container HEALTHCHECK * Docs/Docker: document built-in image HEALTHCHECK * Changelog: note Dockerfile healthcheck probe * Docs/Docker: explain HEALTHCHECK behavior in plain language * Docker: relax HEALTHCHECK interval to 3m --------- Co-authored-by: Vincent Koc <vincentkoc@ieee.org>
dawi369
pushed a commit
to dawi369/davis
that referenced
this pull request
Mar 3, 2026
…penclaw#11478) * Docker: add /healthz-based container HEALTHCHECK * Docs/Docker: document built-in image HEALTHCHECK * Changelog: note Dockerfile healthcheck probe * Docs/Docker: explain HEALTHCHECK behavior in plain language * Docker: relax HEALTHCHECK interval to 3m --------- Co-authored-by: Vincent Koc <vincentkoc@ieee.org>
sachinkundu
pushed a commit
to sachinkundu/openclaw
that referenced
this pull request
Mar 6, 2026
…penclaw#11478) * Docker: add /healthz-based container HEALTHCHECK * Docs/Docker: document built-in image HEALTHCHECK * Changelog: note Dockerfile healthcheck probe * Docs/Docker: explain HEALTHCHECK behavior in plain language * Docker: relax HEALTHCHECK interval to 3m --------- Co-authored-by: Vincent Koc <vincentkoc@ieee.org>
zooqueen
pushed a commit
to hanzoai/bot
that referenced
this pull request
Mar 6, 2026
…penclaw#11478) * Docker: add /healthz-based container HEALTHCHECK * Docs/Docker: document built-in image HEALTHCHECK * Changelog: note Dockerfile healthcheck probe * Docs/Docker: explain HEALTHCHECK behavior in plain language * Docker: relax HEALTHCHECK interval to 3m --------- Co-authored-by: Vincent Koc <vincentkoc@ieee.org>
zooqueen
pushed a commit
to hanzoai/bot
that referenced
this pull request
Mar 6, 2026
…penclaw#11478) * Docker: add /healthz-based container HEALTHCHECK * Docs/Docker: document built-in image HEALTHCHECK * Changelog: note Dockerfile healthcheck probe * Docs/Docker: explain HEALTHCHECK behavior in plain language * Docker: relax HEALTHCHECK interval to 3m --------- Co-authored-by: Vincent Koc <vincentkoc@ieee.org>
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
2 participants
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
Added HEALTHCHECK to the Dockerfile and added debug logging to silent catch blocks in 3 files. No behavior changes.
lobster-biscuit
Why This Matters
HEALTHCHECK lets container orchestrators (Docker Compose, Swarm, K8s) detect an unresponsive gateway and restart it automatically. Uses Node 22 native
fetch()against the/healthendpoint every 30s.Silent catch blocks were making debugging harder. They now log via
getLogger().debug()— crash behavior unchanged.Tests
pnpm buildandpnpm checkpassManual Testing
docker build -t openclaw-test .thendocker inspectshows HEALTHCHECKEvidence
Dockerfile— added HEALTHCHECKsrc/infra/update-startup.ts— log update check failuressrc/agents/model-auth.ts— log auth profile resolution failuressrc/gateway/server-close.ts— log shutdown cleanup failures (pluginServices, configReloader, browserControl)Telegram catch blocks (
withTelegramApiErrorLoggingalready handles logging), media-understanding speculative JSON parse blocks, and cleanup/finallyblocks were intentionally left unchanged.
Greptile Overview
Greptile Summary
This PR adds a Dockerfile HEALTHCHECK that probes the gateway’s
/healthendpoint via Node’s built-infetch(), and replaces a few silentcatch {}blocks withgetLogger().debug(...)calls in the auth profile resolver, gateway shutdown handler, and update check scheduler. The logging changes are localized and keep control flow the same; the main behavioral impact is the new container health probe, which will affect how orchestrators classify the service as healthy/unhealthy.Confidence Score: 3/5
/healthrequires auth under the default container config, the container will flap unhealthy and be restarted despite the gateway running normally.(2/5) Greptile learns from your feedback when you react with thumbs up/down!