Add Enhanced Docker Logging and Optional Notifications#258
Open
St0nefish wants to merge 2 commits intojlesage:masterfrom
Open
Add Enhanced Docker Logging and Optional Notifications#258St0nefish wants to merge 2 commits intojlesage:masterfrom
St0nefish wants to merge 2 commits intojlesage:masterfrom
Conversation
- Stream all MakeMKV output to Docker stdout for better container monitoring - Add optional multi-service notifications via Apprise (80+ services) - Fully backward compatible - all features are opt-in - ENABLE_DOCKER_LOGGING=0 by default, set to 1 to enable stdout logging - Notifications only activate if /config/apprise.yml is configured - Minimal overhead when features are disabled 🤖 Generated with Claude Code Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add Enhanced Docker Logging and Optional Notifications
Overview
This PR adds two opt-in features to improve container monitoring and automation capabilities:
Key Features
docker logsImplementation Details
Enhanced Logging (opt-in)
ENABLE_DOCKER_LOGGING=1to stream MakeMKV output to Docker stdout0(disabled)Notifications (opt-in)
apprise.ymlin/configdirectory to enableNOTIFY_START=1, default:1)New Environment Variables
ENABLE_DOCKER_LOGGING0NOTIFY_START1Files Added
rootfs/usr/bin/makemkv-log-watcher.sh- Monitors MakeMKV log filerootfs/usr/bin/makemkv-monitor.sh- Parses log events and triggers notificationsrootfs/usr/bin/makemkv-notify.py- Apprise notification handlerrootfs/etc/cont-init.d/57-enhanced-logging.sh- Conditional initializationapprise.yml.example- Example notification configurationTesting
Why These Changes?
Example Usage
Enable Docker Logging Only
Enable Notifications
apprise.yml.exampleto/config/apprise.ymlSample Notifications
Rip Started
Rip Complete
Rip Failed (Complete Failure)
Rip Failed (Partial - TV Series)
Notes
app_ShowDebug) is only forced when features are enabled