-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Open
Labels
area/apiIndicates an issue or PR that deals with the API.Indicates an issue or PR that deals with the API.kind/bugCategorizes issue or PR as related to a bug.Categorizes issue or PR as related to a bug.kind/featureCategorizes issue or PR as related to a new feature.Categorizes issue or PR as related to a new feature.
Description
Actual Behaviour
Currently, when marking pod status, we will simply mark the TaskRun that has any of its steps terminated with an non-zero code failed with simply a reason “Failed”. This is covering up the actual Failure reason and inject the failure messages, which are hardly debuggable, into the failure message to be the only place . We should better surface the actual failure reasons based on the conditions at getting failure messages from pod phase and container status.
Expected behaviour
| TaskRunReason when failed | Implication |
|---|---|
| "PodEvicted" | Pod evicted |
| "PodInitFailed" | When any of pod init container failed |
| "PodContainerOOM" | OOM |
| ... | [TBD] for what's returned at extractContainerFailureMessage |
related: #7013
/kind feature
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
area/apiIndicates an issue or PR that deals with the API.Indicates an issue or PR that deals with the API.kind/bugCategorizes issue or PR as related to a bug.Categorizes issue or PR as related to a bug.kind/featureCategorizes issue or PR as related to a new feature.Categorizes issue or PR as related to a new feature.