Skip to content

Conversation

@zootalures
Copy link
Member

@zootalures zootalures commented Feb 10, 2021

This is a quick POC to demonstrate doing docker-daemon level image squashing
This will change the behaviour of image pulls so should be reviewed w.r.t. actual system performance before merging.

@zootalures zootalures force-pushed the oc/squashRuntimeLayers branch from 23c2b50 to 82a9c8d Compare February 10, 2021 20:26
@shaunsmith
Copy link
Contributor

Two issues @owencliffe. The native images don't pass on the build script args to the docker build command so passing "--squash" won't have any effect. The build-native script expects the first arg to be the runtime dir. You've made the first arg "--squash" which will result in failure.

I've got a PR to update the native image builds pending: #246

@zootalures zootalures force-pushed the oc/squashRuntimeLayers branch 2 times, most recently from d21e446 to 9f3ad42 Compare February 11, 2021 12:03
@zootalures zootalures force-pushed the oc/squashRuntimeLayers branch from 9f3ad42 to aeb99b0 Compare February 11, 2021 12:10
Copy link
Contributor

@astromechza astromechza left a comment

Choose a reason for hiding this comment

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

LGTM. although not sure how to build these locally to verify. the ./build.sh script gets stuck on a test somewhere

@zootalures zootalures changed the title Squash docker layers after building WIP: Squash docker layers after building Feb 12, 2021
@zootalures zootalures marked this pull request as draft February 12, 2021 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants