Skip to content

Feature: Add comprehensive Docker development setup - #17

Draft
vishalm wants to merge 2 commits into
openai:mainfrom
vishalm:docker-config
Draft

Feature: Add comprehensive Docker development setup#17
vishalm wants to merge 2 commits into
openai:mainfrom
vishalm:docker-config

Conversation

@vishalm

@vishalm vishalm commented Oct 11, 2025

Copy link
Copy Markdown
  • Add Dockerfile.local for development with hot reload
  • Add docker-compose.local.yml with bind mounts and DNS configuration
  • Add entrypoint.sh for environment variable loading
  • Add .dockerignore to optimize build context
  • Update README.md with Docker development workflow

Features:

  • Hot reload development in Docker container
  • DNS configuration (8.8.8.8, 8.8.4.4) for external API access
  • Environment variable loading from .env.local
  • Port 3000 exposed for local development
  • Source code mounted for real-time changes
  • Bridge networking for optimal connectivity

Usage:

Resolves network connectivity issues with OpenAI API calls in containerized environment.

Vishal Mishra and others added 2 commits October 11, 2025 06:07
- Add Dockerfile.local for development with hot reload
- Add docker-compose.local.yml with bind mounts and DNS configuration
- Add entrypoint.sh for environment variable loading
- Add .dockerignore to optimize build context
- Update README.md with Docker development workflow

Features:
- Hot reload development in Docker container
- DNS configuration (8.8.8.8, 8.8.4.4) for external API access
- Environment variable loading from .env.local
- Port 3000 exposed for local development
- Source code mounted for real-time changes
- Bridge networking for optimal connectivity

Usage:
- docker-compose -f docker-compose.local.yml up --build
- Visit http://localhost:3000 for development

Resolves network connectivity issues with OpenAI API calls in containerized environment.
@seratch

seratch commented Oct 11, 2025

Copy link
Copy Markdown
Member

Thanks for sharing this! It would be great to have a Dockerfile example in this repo in the future, but this one seems a bit more complex than what we’d like to include here. If these changes work best for you, please add the changes to your own fork and use it as your internal template.

@seratch
seratch marked this pull request as draft October 11, 2025 02:33
@vishalm

vishalm commented Oct 11, 2025 via email

Copy link
Copy Markdown
Author

@csfb2f6ytr-cmd

Copy link
Copy Markdown

help

@csfb2f6ytr-cmd

Copy link
Copy Markdown

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.

3 participants