Dockerfile and docker-compose#46
Conversation
Added Dockerfile, docker-compose and updated readme
|
|
||
| services: | ||
| mining-pool: | ||
| image: mauroferra/mining-pool:v1.3.0 |
There was a problem hiding this comment.
I guess we need to change the prefix mauroferra?
There was a problem hiding this comment.
It points to an image that I built with the Dockerfile included, which fetches the code from a release tarball, because the official mining-pool image is quite outdated
There was a problem hiding this comment.
We will not include an unofficial docker image in the reference code. Could you remove the image name and ask people to build the docker manually?
There was a problem hiding this comment.
ok, i can do that, there's the Dockerfile included just for that. I'll make 2 versions, one that uses the current context (useful for building an image from master with latest code) and one that pulls the zip from the latest release, just like it is now.
Will do in a bit today.
- Added Dockerfile for local build - Removed reference to external image - Added instructions to build from source, latest master code and latest release
|
|
||
| 1. From local sources: | ||
| ```shell | ||
| docker build -t mining-pool:latest . |
There was a problem hiding this comment.
with the build: section inside of compose, you can make this automatic 👍
It will name and tag what it builds locally with what you place under image: name:tag
There was a problem hiding this comment.
Thanks!
I added the step and adjusted the readme
Added automatic image build step and adjusted readme
Lbqds
left a comment
Lbqds
left a comment
There was a problem hiding this comment.
Sorry, forgot this, just tested it, LGTM 👍
Fixing up config for compose and shrinking docker image
4 participants
Added a Dockerfile to build an image from the latest Git Release of the mining-pool repo, and a docker-compose file to deploy that image along with redis and a Postgres.
Readme updated to include info about docker deployment.