Skip to content
This repository has been archived by the owner on Nov 19, 2020. It is now read-only.

Dockerfiles add #80

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Dockerfiles add #80

wants to merge 2 commits into from

Conversation

WeslyG
Copy link

@WeslyG WeslyG commented Jun 29, 2018

Added Dockerfiles to docker folder (centos).
Took out all configurations in ENV and added them to docker-compose
One service - one container.
Left only the most necessary configuration, additional configuration commented.

Containers upload on hub.docker.com here

@codecov
Copy link

codecov bot commented Jun 29, 2018

Codecov Report

Merging #80 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #80   +/-   ##
=======================================
  Coverage   30.91%   30.91%           
=======================================
  Files         145      145           
  Lines        9271     9271           
=======================================
  Hits         2866     2866           
  Misses       6304     6304           
  Partials      101      101

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 691b052...ea93f96. Read the comment docs.

@AlexAkulov
Copy link
Collaborator

Maybe you can use "alpine" image instead "centos" or even "scratch" if you try.

@WeslyG WeslyG closed this Jul 2, 2018
@WeslyG WeslyG reopened this Jul 2, 2018
@WeslyG
Copy link
Author

WeslyG commented Jul 2, 2018

Moved the containers into the alpine while retaining the functionality of the settings via env. 11mb vs 84mb on centOs

@AlexAkulov
Copy link
Collaborator

armv7? Is it work?

@WeslyG
Copy link
Author

WeslyG commented Jul 2, 2018

Yes, it works, and that's the only way I managed to run bitfan on alpine. On amd64 everything falls with the error "no such file or directory"

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants