Merge lp://staging/~ricardokirkner/isitdeployable/smaller-docker-1 into lp://staging/isitdeployable
Proposed by
Ricardo Kirkner
| Status: | Merged |
|---|---|
| Approved by: | Ricardo Kirkner |
| Approved revision: | 232 |
| Merge reported by: | Otto Co-Pilot |
| Merged at revision: | not available |
| Proposed branch: | lp://staging/~ricardokirkner/isitdeployable/smaller-docker-1 |
| Merge into: | lp://staging/isitdeployable |
| Prerequisite: | lp://staging/~ricardokirkner/isitdeployable/non-root-containers |
| Diff against target: |
148 lines (+50/-25) 7 files modified
Dockerfile (+6/-6) Dockerfile.test (+9/-0) README.k8s (+25/-19) dependencies-devel.txt.docker (+2/-0) dependencies.txt.docker (+6/-0) django_project/settings_base.py (+1/-0) requirements.txt (+1/-0) |
| To merge this branch: | bzr merge lp://staging/~ricardokirkner/isitdeployable/smaller-docker-1 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Daniel Manrique (community) | Approve | ||
|
Review via email:
|
|||
Commit message
reduce container size by installing as few system dependencies as possible
Description of the change
It seems like most dependencies in dependencies.txt were not really (strictly) necessary for running the service.
Installing only the strictly necessary packages reduced the size of the built container from ~845M -> ~488M (43% reduction).
Further improvement is likely possible, as adding the software-
To post a comment you must log in.
Revision history for this message
| Ricardo Kirkner (ricardokirkner) | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
This is awesome, many thanks!
the magic software-
Also, python-pygit2 is available in xenial archive, so if/when we move to xenial, the ppa requirement may go away altogether.
Finally, proposed one possible further reduction of cruft. But this can go as-is.