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

Fallback to slow behavior if writing to daemon fails with performance optimization #80

Closed
ekcasey opened this issue Dec 3, 2020 · 0 comments · Fixed by #81
Closed
Labels
status/ready type/bug Something isn't working

Comments

@ekcasey
Copy link
Member

ekcasey commented Dec 3, 2020

We currently omit base image layers when writing to the docker daemon in order to improve performance. However some container runtimes like Podman that implement the daemon API do not allow an image load with missing layers. Therefore if our first attempt at writing fails, we should fall back on the slow/complete behavior and provide all layer tars.

context: buildpacks/pack#925

@ekcasey ekcasey added type/bug Something isn't working status/ready labels Dec 3, 2020
ekcasey added a commit to buildpacks/lifecycle that referenced this issue Dec 10, 2020
Falls back to slower behavior if the daemon does not accept empty layers. This allows the lifecycle to be compatible with podman which requires all layers to be preset during a docker load. buildpacks/imgutil#80

Fixes lifecycle errors in analyze, export, and rebase resulting from a bug in our daemon save logic buildpacks/imgutil#60

Signed-off-by: Emily Casey <ecasey@vmware.com>
ekcasey added a commit to buildpacks/lifecycle that referenced this issue Dec 10, 2020
Falls back to slower behavior if the daemon does not accept empty layers. This allows the lifecycle to be compatible with podman which requires all layers to be preset during a docker load. buildpacks/imgutil#80

Fixes "manifest.json had unexpected number of entries"  errors in analyze, export, and rebase resulting from a bug in our daemon save logic buildpacks/imgutil#63

Signed-off-by: Emily Casey <ecasey@vmware.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/ready type/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant