-
Notifications
You must be signed in to change notification settings - Fork 22
Add dockerfile, docker-compose.yaml #228
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
Conversation
Would probably add the same waiting strategy you have for postgres for redis. I'm curious if it wouldn't be easier for us to just add |
|
Having the docker compose is quite useful, I believe it should now work with our standard docker file so I'm removing yours + the entrypoint script. |
* initial commit, mostly working * Delete entrypoint.sh * Delete Dockerfile --------- Co-authored-by: George <george@cerebralab.com>
* initial commit, mostly working * Delete entrypoint.sh * Delete Dockerfile --------- Co-authored-by: George <george@cerebralab.com>
* initial commit, mostly working * Delete entrypoint.sh * Delete Dockerfile --------- Co-authored-by: George <george@cerebralab.com>
There are still some issues & work remaining, but would you all be generally interested in something like this to use docker/docker-compose to simplify the install steps?
I'm getting some errors but the app mostly comes up - I'm not sure if it's expected to fully work before #194 is merged, so I'll re-test after that, but let me know if you have any thoughts/questions/feedback whenever.