Skip to content

Add bounded read-only RCON server health checks - #624

Merged
fank merged 2 commits into
masterfrom
codex/add-server-healthcheck
Nov 2, 2026
Merged

fank merged 2 commits into
masterfrom
codex/add-server-healthcheck

Conversation

@fank

@fank fank commented Nov 2, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Add a local, read-only RCON health check to regular and rootless images.
  • Require a valid response within five seconds; missing credentials, authentication failure and unresponsive servers report unhealthy.
  • Document Docker health status, tuning and disabling the check. No public RCON or HTTP endpoint is needed.

Closes #532.
Closes #476.

Validation

  • All 21 existing regression methods, ShellCheck and whitespace checks pass.
  • The health-check implementation is unchanged from the previously tested regular/rootless images: healthy startup, missing/incorrect credentials, and recovery were verified without automatic restarts.
  • Removed the added Python test file following maintainer feedback. The PR now changes four files with 25 added lines.

@fank
fank marked this pull request as ready for review November 2, 2026 18:38
@fank
fank merged commit 328b62e into master Nov 2, 2026
4 checks passed
@fank
fank deleted the codex/add-server-healthcheck branch November 3, 2026 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Docker healthcheck server health

1 participant