Skip to content

fix(docker): add procps to all Dockerfiles for HEALTHCHECK#352

Merged
thepagent merged 1 commit intoopenabdev:mainfrom
chaodu-agent:fix/dockerfile-codex-add-procps
Apr 15, 2026
Merged

fix(docker): add procps to all Dockerfiles for HEALTHCHECK#352
thepagent merged 1 commit intoopenabdev:mainfrom
chaodu-agent:fix/dockerfile-codex-add-procps

Conversation

@chaodu-agent
Copy link
Copy Markdown
Collaborator

@chaodu-agent chaodu-agent commented Apr 15, 2026

Problem

All five Dockerfiles use pgrep -x openab in their HEALTHCHECK, but procps (which provides pgrep) is not included in the node:22-bookworm-slim base image. This means health checks have been silently failing across all container variants.

Solution

Add procps to the existing apt-get install line in the runtime stage of every Dockerfile.

Files Changed

  • Dockerfile
  • Dockerfile.claude
  • Dockerfile.codex
  • Dockerfile.copilot
  • Dockerfile.gemini

Minimal, focused fix — only addresses the missing pgrep dependency.

@chaodu-agent chaodu-agent requested a review from thepagent as a code owner April 15, 2026 00:58
@github-actions github-actions bot added the closing-soon PR missing Discord Discussion URL — will auto-close in 3 days label Apr 15, 2026
@github-actions
Copy link
Copy Markdown

⚠️ This PR is missing a Discord Discussion URL in the body.

All PRs must reference a prior Discord discussion to ensure community alignment before implementation.

Please edit the PR description to include a link like:

Discord Discussion URL: https://discord.com/channels/...

This PR will be automatically closed in 3 days if the link is not added.

All Dockerfiles use 'pgrep -x openab' in their HEALTHCHECK but procps
was not installed in the node:22-bookworm-slim base image, causing
health checks to always fail silently.

Add procps to the apt-get install line in:
- Dockerfile
- Dockerfile.claude
- Dockerfile.codex
- Dockerfile.copilot
- Dockerfile.gemini
@chaodu-agent chaodu-agent force-pushed the fix/dockerfile-codex-add-procps branch from d1a1bed to d89c3e6 Compare April 15, 2026 01:02
@chaodu-agent chaodu-agent changed the title fix(docker): add procps to Dockerfile.codex for HEALTHCHECK fix(docker): add procps to all Dockerfiles for HEALTHCHECK Apr 15, 2026
@thepagent thepagent merged commit fbb265d into openabdev:main Apr 15, 2026
6 of 8 checks passed
@thepagent thepagent removed the closing-soon PR missing Discord Discussion URL — will auto-close in 3 days label Apr 15, 2026
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.

2 participants