Merge pull request #100 from drone-runners/revert-99-CI-10637

Revert "(CI-10637) error to console placeholder/step failure"
drone-runners · Dec 12, 2023 · b57d0a9 · b57d0a9
2 parents d44ac65 + 1e46b08
commit b57d0a9
Showing 1 changed file with 2 additions and 7 deletions.
diff --git a/engine/podwatcher/watcher.go b/engine/podwatcher/watcher.go
@@ -285,17 +285,12 @@ func (pw *PodWatcher) notifyClients(c *containerWatchInfo) {
 	case stepStateRunning, stepStateFinished:
 		pw.notifyClientsContainerChange(c)
 	case stepStatePlaceholderFailed, stepStateFailed:
-		failedContainer := FailedContainerError{
+		pw.notifyClientsError(c, FailedContainerError{
 			container: c.id,
 			exitCode:  c.exitCode,
 			reason:    c.reason,
 			image:     c.image,
-		}
-		logrus.WithField("pod", pw.podName).
-			WithField("container", c.id).
-			WithField("stepState", c.stepState).
-			Error(failedContainer.Error())
-		pw.notifyClientsError(c, failedContainer)
+		})
 	}
 }