Skip to content

Commit

Permalink
Merge pull request #739 from dzoeteman/build-args-cache
Browse files Browse the repository at this point in the history
Reverted not including build args in cache key
  • Loading branch information
priyawadhwa authored Aug 16, 2019
2 parents a111681 + c425f02 commit 48e25aa
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions pkg/executor/build.go
Original file line number Diff line number Diff line change
Expand Up @@ -191,6 +191,7 @@ func (s *stageBuilder) optimize(compositeKey CompositeCache, cfg v1.Config) erro
func (s *stageBuilder) build() error {
// Set the initial cache key to be the base image digest, the build args and the SrcContext.
compositeKey := NewCompositeCache(s.baseImageDigest)
compositeKey.AddKey(s.opts.BuildArgs...)

// Apply optimizations to the instructions.
if err := s.optimize(*compositeKey, s.cf.Config); err != nil {
Expand Down

0 comments on commit 48e25aa

Please sign in to comment.