Dockerify#7
Conversation
|
Thanks for putting this together. Sadly I'm not a docker expert, but a couple of thoughts:
|
|
Sure, I can update the README and also add a new data volume to store the db. To run it with Docker (which was useful in my local testing for my other two or so PRs): |
|
@tdeck Thoughts on this? I just rebased but wanted to check with you before I continue this work. |
|
Yep, looks good in general. I just still have the inline question about that exposed port, and also I wonder what will happen with the team database if people use this configuration. I thought that generally things in a docker container were treated as ephemeral, and you were supposed to separate out the data. But I'm not really a docker user and I assume it depends on your platform. |
Note: I set TEAM_DB_FILE=/var/run/data/teambot.db in my `.env`.
| volumes: | ||
| - .:/src | ||
| expose: | ||
| - "8000" |
|
@tdeck Not quite, been swapped here lately. Though I've not forgotten and will plan to make time for this ASAP. |
|
@tdeck I cannot seem to find your inline comment. But I think I know what you're asking about: No, we probably don't need to expose anything. I just copy pasted that from another docker-based project I'm working on and must've forgotten to remove it. Good catch! |
|
No hurry on my end, just want to make sure I wasn't neglecting anything. Let me know when you want me to take another look. |
TODOs: