Skip to content

Conversation

@adamoutler
Copy link
Collaborator

@adamoutler adamoutler commented Jan 23, 2026

Added necessary capabilities for root-entrypoint operations.

Summary by CodeRabbit

  • Chores
    • Enhanced Docker Compose service configuration with necessary Linux capabilities to properly manage file permissions and user privileges during container startup.

✏️ Tip: You can customize this high-level summary in your review settings.

Added necessary capabilities for root-entrypoint operations.
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 23, 2026

📝 Walkthrough

Walkthrough

Updates Docker Compose configuration documentation by adding three Linux capabilities (CHOWN, SETUID, SETGID) to service capabilities. These capabilities enable the root-entrypoint script to manage filesystem permissions on /data and /tmp directories and facilitate privilege dropping to a non-root user/group.

Changes

Cohort / File(s) Summary
Docker Configuration Documentation
docs/DOCKER_COMPOSE.md
Adds CHOWN, SETUID, and SETGID Linux capabilities to Docker Compose service configuration for permission management and privilege escalation control

Poem

🐰 In Docker's realm where permissions take flight,
Three powers granted to set things right,
CHOWN and SETUID dance with grace,
SETGID joins the privileged space,
Root can manage, then step away bright! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 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 Docker Compose capabilities for the root-entrypoint process, which is the primary modification in the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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

@jokob-sk jokob-sk merged commit 9da1d2a into netalertx:main Jan 23, 2026
5 checks passed
@adamoutler adamoutler mentioned this pull request Jan 28, 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