Skip to content
This repository was archived by the owner on Jun 3, 2025. It is now read-only.

Conversation

@mzihlmann
Copy link

@mzihlmann mzihlmann commented May 31, 2025

Description

So far ADD instructions were not cacheable, only COPY instructions. This is not really a blocker as most operations can and should be done with COPY instead, but nevertheless it's confusing.

With this change we implement caching analogous to COPY instructions. No special precautions for tar unpacking etc has to be taken as they only affect the initial build, not the rebuild from cache.

Submitter Checklist

  • Includes unit tests
  • Adds integration tests if needed.

See the contribution guide for more details.

Reviewer Notes

  • The code flow looks good.
  • Unit tests and or integration tests added.

Release Notes

Examples of user facing changes:
- ADD learned to cache it's output layer

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant