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

Dockerfile + README.md usage #27

Closed
wants to merge 3 commits into from
Closed

Conversation

dexafree
Copy link
Contributor

@dexafree dexafree commented Mar 2, 2018

This PR adds a Dockerfile that will build Shiori based on the official Go alpine image (by using the alpine version, the resulting image will be smaller).

Comparison between using the standard image (left) vs the Alpine one (right):

Comparison

It also adds a little "Usage with Docker" section to the Readme file that will help the users self-host their own shiori containers, with the ability to persist the database across container recreations.

@maxmeyer
Copy link
Contributor

maxmeyer commented Mar 5, 2018

I suggest to use multi stage builds instead: https://docs.docker.com/develop/develop-images/multistage-build/#name-your-build-stages. There's no need for a full "go"-aware alpine image for that. The link contains a suggestion about how to build go projects with that docker feature.

@maxmeyer maxmeyer mentioned this pull request Mar 5, 2018
@maxmeyer
Copy link
Contributor

maxmeyer commented Mar 5, 2018

#11 and #27 trying to solve the same problem.

This was referenced Mar 9, 2018
@RadhiFadlillah
Copy link
Collaborator

Closed since #66 has been merged.

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.

None yet

3 participants