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

Optimize the size of the built image #305

Closed
wants to merge 1 commit into from

Conversation

soulteary
Copy link

Inspired by #278 , thanks @n8225

This PR only optimized the final docker image size, eg:

shiori                                        latest                           553f858e15cb   9 minutes ago    13.5MB
radhifadlillah/shiori                         latest                           69e24681b088   8 months ago     422MB

@ghost
Copy link

ghost commented May 7, 2021

You can further reduce by packing with UPX:

https://gitlab.com/equalos/skaffold/-/blob/master/shiori/docker/shiori/Dockerfile

@soulteary
Copy link
Author

You can further reduce by packing with UPX:

https://gitlab.com/equalos/skaffold/-/blob/master/shiori/docker/shiori/Dockerfile

good idea :)

pyunramura added a commit to pyunramura/shiori that referenced this pull request Jan 26, 2022
@fmartingr
Copy link
Member

Hey @soulteary, thanks a lot for the contribution! There has been a reworks of some parts of the application and another contributor already reduced the container image quite a lot. I think this PR is a bit redundant now, but if you still feel like there's room for a smaller image size, please open a new PR and we will take a look a it. Thank you!

@fmartingr fmartingr closed this Feb 16, 2022
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

2 participants