Skip to content

Conversation

pataquets
Copy link
Contributor

Optimize final image build size by grouping commands in the same build step and deleting unneeded package installation files. Add non-interactive env vars.

…d step and deleting unneeded package installation files. Add non-interactive env vars.
Copy link
Member

@itamarhaber itamarhaber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@itamarhaber
Copy link
Member

@pataquets one small thing - could you also test whether adding no-install-recommends to apt reduces the build time and at the same time does not break it?

@pataquets
Copy link
Contributor Author

@itamarhaber : Done. Works for me, it didn't broke.
Also, I've done a few improvements in separate commits. Mainly changing -yy for -y, since some nasty errors might be hidden otherwise. Make it a bit more "fail fast, fail loud" in case something is wrong when building.
I've set up an automated build at https://hub.docker.com/r/pataquets/memtier_benchmark ease testing.

@itamarhaber itamarhaber merged commit 050f29c into RedisLabs:master Jun 19, 2019
@itamarhaber
Copy link
Member

Thanks @pataquets - merged.

yaacovhazan-Redislabs pushed a commit that referenced this pull request Mar 18, 2020
* Optimize final image build size by grouping commands in the same build step and deleting unneeded package installation files. Add non-interactive env vars.

* Add non-interactive env vars to builder apt-get. Readability.

* apt-get switches: be less forceful to allow detection of potentially harmful build errors.

* Add '--no-install-recommends' switch to apt-get command to reduce build time and make image smaller (#70).

* Add testing Docker Compose manifests.

* Dockerfile: Get source files from local build context instead of downloading from git.

* Docker Compose example manifests: Build from local context. Uncomment example options.

* Readme: Document Docker Compose manifest usage.
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.

2 participants