Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add support for docker dev #100

Open
wants to merge 31 commits into
base: main
Choose a base branch
from

Conversation

bhardwajRahul
Copy link

This PR aims at adding docker dev support in bolt.

@bhardwajRahul bhardwajRahul changed the title Add support for docker dev in bolt feat: add support for docker dev in bolt Oct 7, 2024
@d3lm d3lm force-pushed the main branch 2 times, most recently from f66de27 to 849880e Compare October 7, 2024 08:50
@d3lm d3lm changed the title feat: add support for docker dev in bolt feat: add support for docker dev Oct 7, 2024
@bhardwajRahul
Copy link
Author

@d3lm hey,
Why didn't you guys added this repo to hacktoberfest.

@geekyharsh05
Copy link

@bhardwajRahul It will be better to add a docker compose file for dev mode?

Dockerfile Outdated Show resolved Hide resolved
Copy link

@geekyharsh05 geekyharsh05 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How will you run this? add a compose file it will be better.

@bhardwajRahul
Copy link
Author

bhardwajRahul commented Oct 11, 2024

How will you run this? add a compose file it will be better.
@geekyharsh05
It is single service and why does it makes sense to have a docker-compose for a single independent service. Moreover, Docker-compose makes sense when your service is dependent on other components like Databases etc. Its a one liner deployment for development purpose.
I work for Digitalocean and I am using docker image( https://hub.docker.com/r/mickysharam/bolt-ai) deployed over my App platform by passing in all the environment variables. You are free to use any PAAS.

docker run -p 5173:5173 --env-file .env.local mickysharam/bolt-ai

I hope it makes sense now.

@Felitendo
Copy link

How will you run this? add a compose file it will be better.
@geekyharsh05
It is single service and why does it makes sense to have a docker-compose for a single independent service. Moreover, Docker-compose makes sense when your service is dependent on other components like Databases etc. Its a one liner deployment for development purpose.
I work for Digitalocean and I am using docker image( https://hub.docker.com/r/mickysharam/bolt-ai) deployed over my App platform by passing in all the environment variables. You are free to use any PAAS.

docker run -p 5173:5173 --env-file .env.local mickysharam/bolt-ai

I hope it makes sense now.

Well that would work, but I don't trust you to keep the docker image updated. I think stackblitz should make their own docker image

@schaveyt
Copy link

I was actually working with this very docker image as well and encountered issues.

My 2 cents: we take a que from OpenWebUI and get everything to be able to run in a self-contained docker image

This then can either be run at home or in the office on a container...or where ever

So first, a focus to eliminate Wrangler and Cloudflare...then use a docker-compose.yml and maybe a bash script (unix) and poweshell (windows) to build, start, stop, restart the container.

I can then make a systemd service file to install as a service as well.

My big issue ..I can get the dang thing to work on windows
I think if we can focus, first to eliminate the damn cloudflare wrangler crap and to just be

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.