Update internal/directives/simple_engine_promote.go

akuity · Dec 5, 2024 · 99a990a · 99a990a
1 parent 8db25c0
commit 99a990a
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/internal/directives/simple_engine_promote.go b/internal/directives/simple_engine_promote.go
@@ -131,7 +131,7 @@ func (e *SimpleEngine) executeSteps(
 		}
 
 		// Treat errors and logical failures the same for now.
-		// TODO(krancour): These may be handled differently in the future.
+		// TODO(krancour): In the future, we should fail without retry for logical failures and unrecoverable errors and retry only those errors with a chance of recovery.
 		if stepExecMeta.Status != kargoapi.PromotionPhaseRunning {
 			stepExecMeta.ErrorCount++
 			// Check if the error threshold has been met.