Skipped tasks shown as pending

[larhauga]

Describe the bug

Pipelines with tasks that are skipped (like due to non master branch run) is observed to have the state of pending even though it has been skipped. This issue came to us from end-users, which did not understand why the task was still pending.

Expected behaviour

We expect the status of the skipped task to be Skipped; not Pending.

Steps to reproduce the bug

In this case we have a when condition, that skipps this task if branch is main or master.

...
    - name: promote
      runAfter:
      - mvn-build
      taskRef:
        kind: ClusterTask
        name: # taskname
      when:
      - input: $(params.gitbranchname)
        operator: in
        values:
        - master
        - main
...

Environment details

• Kubernetes Platform: EKS

• Kubernetes or OpenShift version: 1.21

• Install mode (if on OpenShift): no

• Cloud-provider/provisioner: EKS

• Versions:
  • Tekton Dashboard: 0.23.0
  • Tekton Pipelines: 0.31.0
  • Tekton Triggers: 0.18.0
• Install namespaces:
  • Tekton Dashboard: tekton-pipelines
  • Tekton Pipelines:
  • Tekton Triggers:

Additional Info

[AlanGreene]

Thanks for raising this issue @larhauga. We're currently considering a redesign of this page to improve display of a number of things including when expressions, retries, and more.

[tekton-robot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale with a justification.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close with a justification.
If this issue should be exempted, mark the issue as frozen with /lifecycle frozen with a justification.

/lifecycle stale

Send feedback to tektoncd/plumbing.

[AlanGreene]

This is something we'll be addressing as part of the PipelineRun details page redesign which is currently in progress. Freezing so it doesn't get auto closed.

/lifecycle frozen

[AlanGreene]

When we address this we should review tektoncd/pipeline#4829 and TEP-0103. This makes additional useful information available regarding the reason for a task being skipped.

[AlanGreene]

Hoping to revisit this soon. Related to #1445

[AlanGreene]

Blocked by #3493 which in currently is progress. It affects pretty much the entire front-end codebase so we've paused functional changes util that's complete, ETA ~end of month.

Once that's done, this is among the next batch of UI updates planned.

[AlanGreene]

#3651 improves the display of skipped tasks, making it clearer to users that these are skipped and not actually pending. The corresponding when expression or other relevant reason is displayed in the taskrun status, and the log container clearly identifies that no logs will be displayed due to the task being skipped.

This will be available in the next nightly release, and included in Dashboard v0.52.0 which is expected in the next week or so.

Further improvements will be made as part of #2306

[AlanGreene]

This is now available in https://github.com/tektoncd/dashboard/releases/tag/v0.52.0