Open
Conversation
This commit adds a comprehensive Docker deployment system to make SubGen
production-ready and easier to deploy:
New Files:
- .env.example: Comprehensive environment variable template with all options
documented and organized by category
- docker-compose.yml: Modernized Docker Compose file (v2 format) with:
* Build context for local builds
* Health checks
* Better environment variable handling with defaults
* Improved documentation
- docker-compose.gpu.yml: GPU-specific compose file with NVIDIA GPU support
- DOCKER_DEPLOYMENT.md: Complete deployment guide covering:
* Prerequisites and setup instructions
* CPU and GPU deployment options
* Media server configuration (Plex, Jellyfin, Emby, Bazarr)
* Model selection guide
* Troubleshooting and best practices
- Makefile: Simple commands for deployment (make start, make logs, etc.)
- QUICKSTART.md: Quick start guide for getting running in 5 minutes
Improvements:
- Removed deprecated docker-compose version field
- Added build contexts to allow local building
- Added health checks for monitoring
- Better volume and path documentation
- Environment variables now use defaults (${VAR:-default})
- Comprehensive documentation for all deployment scenarios
This makes SubGen production-ready while maintaining backward compatibility
with existing deployments using the mccloud/subgen images.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…L5hfTMsr7QDGCf7znt Add production-ready Docker deployment system
Contributor
|
There is already a example compose file, why would you add that? https://github.com/McCloudS/subgen/blob/main/docker-compose.yml |
Author
|
Because the one you have makes no note of GPU usage, or well, a lot but I really, frankly, don't care at the end of the day. I integrated bazarr-compatible whisper with another project a while back |
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
3 participants
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.
Hey,
I used Claude Code to add a docker deployment system. I did obviously use AI, but it is just a docker system, and I use it myself