Epic: User-facing Error Handling Improvement 🧹

[JeromeJu]

Context

This issue tracks the improvement of error handling for pipeline as an epic. This includes improving the user-facing error messages, differentiating expected/ runtime errors and cleanups. It aims to improve the debug experience involving cluster operators, service providers and end users building on Tekton.

Note that the users include service provider and cluster operator.

Reference of errors for stakeholders:

• Logging error reference for cluster operators and service providers.
• User-facing error cleanup (to examine whether the existing messages are sufficient).

Work plan:

Differentiating user versus system errors:

• Distinction of expected failures and other errors in tasks and pipelines #6859
  • Label user error for failure PipelineRun Status Message #7475
  • Label user error for failed TaskRunStatus message #7543
• TEP-0151: Error attribution via condition status

Better User-facing Error Reference and Messages:

• deduplicate and update the taskRun status to provide meaningful info  #7013
  • Cleanup error message for Step container status #7405
• Decompose conflated user-facing PipelineRunStatus/ TaskRunStatus failure reasons #7434
• Error sweep: complete user-facing error messages contexts #7474
• Error sweep: add more context for PipelineRunCouldntGetPipeline error message #7403
• Error sweep: fix error messages for timing out Runs #7572
• Replace PipelineRunReasonFailedValidation with more granular reasons #7417
• Error sweep: Move TaskRun Reasons in pkg/pod to pkg/apis #7406

Validation

• ViaField/ ViaFieldIndex does not propagate validation errors along the specified field path access #7258 for Validation Error

Fix edge cases:

• Fix: do not fail TaskRun for concurrent modification errors #7467

Future work identified

• TEP-0151: Error attribution via condition status
• Enhance user-facing Condition/ContainerTerminated Reasons with backwards compatibility #7539

Nice-to-have

Go style cleanups:

• [Go Style] Inconsistency of error message capitalizing #7266

• As one of the followups Fix capitalization inconsistency in reconciler #7318 (comment), we might want to have godot linter enabled.

• Better Container termination status when failing TaskRun for Image pull errors #7385

[tekton-robot]

@JeromeJu: The label(s) kind/area/epic cannot be applied, because the repository doesn't have them.

In response to this:

/kind area/epic

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.