Skip to content

Conversation

@J4ker
Copy link

@J4ker J4ker commented Nov 4, 2025

install procps for worker container healthcheck - pgrep

Summary by CodeRabbit

  • Chores
    • Updated container image base packages to include the procps utility, ensuring additional system monitoring tools are available in the runtime environment.

install procps for worker container healthcheck - pgrep

Signed-off-by: J4ker <[email protected]>
@coderabbitai
Copy link

coderabbitai bot commented Nov 4, 2025

Walkthrough

The Dockerfile's base package installation command was updated to include the procps package in the apt-get install list, adding process utilities to the container image during the base layer setup phase.

Changes

Cohort / File(s) Summary
Devcontainer Dockerfile
\.devcontainer/Dockerfile
Added procps to the apt-get install list in the Dockerfile's package installation RUN step. No other changes to control flow or configuration.

Sequence Diagram(s)

N/A

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

  • Single-line dependency addition in one Dockerfile; minimal risk.

Poem

🐰 I hopped into the build and found a tiny tweak,
procps in the list — a helpful tool to seek.
A soft drumroll of processes now at play,
The image grows wiser, in a gentle way. ✨

Pre-merge checks and finishing touches

✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main change: adding procps to the Dockerfile package installation to support healthcheck functionality.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1ceb526 and f3ee2b8.

📒 Files selected for processing (1)
  • .devcontainer/Dockerfile (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)
  • GitHub Check: ci / scan_ruby
  • GitHub Check: ci / test
  • GitHub Check: ci / lint
  • GitHub Check: ci / scan_js
  • GitHub Check: Cursor Bugbot
🔇 Additional comments (1)
.devcontainer/Dockerfile (1)

20-20: Clean addition of procps for healthcheck support.

The addition of procps to the package list is correct and aligns with the PR objective. The syntax is proper, and the package will provide utilities like pgrep needed for container healthchecks.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Collaborator

@jjmata jjmata left a comment

Choose a reason for hiding this comment

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

I've done this at least once! ;-)

Added it also to the dev container Dockerfile which for some reason it's got a different list of installed packages. Want to sync them up/test, @J4ker?

CodeSpaces actually stopped working recently, wonder if related.

@J4ker
Copy link
Author

J4ker commented Nov 4, 2025

I have very limited skill in this one, I have no idea what devcontainer does

@jjmata
Copy link
Collaborator

jjmata commented Nov 4, 2025

I have very limited skill in this one, I have no idea what devcontainer does

Don't worry ... that can come later. It's basically a way to wrap docker containers to help set up development environments.

@jjmata jjmata merged commit 749bcda into we-promise:main Nov 4, 2025
7 checks passed
@jjmata jjmata added the infra/tooling Tools related to building/operating sure, not sure itself label Nov 7, 2025
alessiocappa pushed a commit to alessiocappa/sure that referenced this pull request Nov 16, 2025
* Update Dockerfile - procps

install procps for worker container healthcheck - pgrep

Signed-off-by: J4ker <[email protected]>

* Good idea, adding it to dev container

---------

Signed-off-by: J4ker <[email protected]>
Co-authored-by: Juan José Mata <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infra/tooling Tools related to building/operating sure, not sure itself

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants