handle all terminal states when processing logs

The prior check for whether the results Object had completed did not
account for objects which had been cancelled.  Rather then check every
conceivable terminal state, this commit instead makes sure it is not
in the Unknown state, which means it has not terminated.

rh-pre-commit.version: 2.1.0
rh-pre-commit.check-secrets: ENABLED

pkg/watcher/reconciler/dynamic/dynamic.go

@@ -285,7 +285,7 @@ func (r *Reconciler) sendLog(ctx context.Context, o results.Object) error {
 		(GVK.Kind == "TaskRun" || GVK.Kind == "PipelineRun") &&
 		condition != nil &&
 		condition.Type == "Succeeded" &&
-		(condition.Reason == "Succeeded" || condition.Reason == "Completed" || condition.Reason == "Failed") {
+		!condition.IsUnknown() {
 
 		rec, err := r.resultsClient.GetLogRecord(ctx, o)
 		if err != nil {