Skip to content

Remove logging driver from Docker Compose config#1483

Merged
jokob-sk merged 1 commit intonetalertx:mainfrom
adamoutler:patch-13
Feb 3, 2026
Merged

Remove logging driver from Docker Compose config#1483
jokob-sk merged 1 commit intonetalertx:mainfrom
adamoutler:patch-13

Conversation

@adamoutler
Copy link
Member

@adamoutler adamoutler commented Feb 3, 2026 •
edited by coderabbitai bot
Loading

Removed the line specifying json driver, thus allowing system defaults. Synology does not support json driver.

Summary by CodeRabbit

  • Documentation
    • Simplified Docker Compose logging configuration settings in documentation.

@adamoutler
Remove logging driver from Docker Compose config
b87a8d6 
Allow system defaults. Synology does not support json driver.
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 3, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

The logging driver specification was removed from the Docker Compose configuration file, while preserving the remaining logging options for max-size and max-file settings. This is a minor configuration adjustment to the baseline Docker Compose setup.

Changes

Cohort / File(s) Summary
Docker Compose Configuration
docs/DOCKER_COMPOSE.md		 Removed the driver: "json-file" logging driver specification from the logging section; other logging constraints remain unchanged.

Poem

🐰 A logging driver takes its final bow,
Simplified config, streamlined now,
One line less, the setup flows,
Cleaner compose wherever it goes! ✨

🚥 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 describes the main change: removing a logging driver specification from the Docker Compose configuration.
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.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • ✨ 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Failure to add the new IP will result in interrupted reviews.

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.

@jokob-sk jokob-sk merged commit 250b5a3 into netalertx:main Feb 3, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@adamoutler @jokob-sk