Fixes #1544 (#1597)

larsgroeber · web-flow · commit 0477900febfa · 2021-03-31T10:17:26.000-07:00
* Specifying a tarPath will push the image as well
diff --git a/pkg/executor/push.go b/pkg/executor/push.go
@@ -236,7 +236,10 @@ func DoPush(image v1.Image, opts *config.KanikoOptions) error {
 		for _, destRef := range destRefs {
 			tagToImage[destRef] = image
 		}
-		return tarball.MultiWriteToFile(opts.TarPath, tagToImage)
+		err := tarball.MultiWriteToFile(opts.TarPath, tagToImage)
+		if err != nil {
+			return errors.Wrap(err, "writing tarball to file failed")
+		}
 	}
 
 	if opts.NoPush {

Original file line number	Diff line number	Diff line change
`@@ -236,7 +236,10 @@ func DoPush(image v1.Image, opts *config.KanikoOptions) error {`
`236`	`236`	`for _, destRef := range destRefs {`
`237`	`237`	`tagToImage[destRef] = image`
`238`	`238`	`}`
`239`		`- return tarball.MultiWriteToFile(opts.TarPath, tagToImage)`
	`239`	`+ err := tarball.MultiWriteToFile(opts.TarPath, tagToImage)`
	`240`	`+ if err != nil {`
	`241`	`+ return errors.Wrap(err, "writing tarball to file failed")`
	`242`	`+ }`
`240`	`243`	`}`
`241`	`244`
`242`	`245`	`if opts.NoPush {`