Merge pull request #931 from asottile/clean_up_artifacts_on_error

Merged by openshift-bot
openshift · Oct 12, 2018 · 2a78342 · 2a78342
2 parents 4053137 + ea98870
commit 2a78342
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/pkg/build/strategies/sti/sti.go b/pkg/build/strategies/sti/sti.go
@@ -513,6 +513,11 @@ func (builder *STI) Save(config *api.Config) (err error) {
 		extractErr := builder.tar.ExtractTarStream(artifactTmpDir, outReader)
 		io.Copy(ioutil.Discard, outReader) // must ensure reader from container is drained
 		builder.result.BuildInfo.Stages = api.RecordStageAndStepInfo(builder.result.BuildInfo.Stages, api.StageRetrieve, api.StepRetrievePreviousArtifacts, startTime, time.Now())
+
+		if extractErr != nil {
+			builder.fs.RemoveDirectory(artifactTmpDir)
+		}
+
 		return extractErr
 	}