Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PipelineRun fails to timeout properly in v0.63.0 (hits PipelineRunCouldntTimeOut state) #8230

Open
hrivera-ntap opened this issue Sep 2, 2024 · 1 comment · May be fixed by #8236
Open
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@hrivera-ntap
Copy link

Expected Behavior

Creating a pipelinerun with a timeout set that has multiple tasks, where some complete before the timeout expires should properly timeout.

Actual Behavior

The pipelinerun resource hits a "PipelineRunCouldntTimeout" state with the following error:

  Normal   PipelineRunCouldntTimeOut  10s                PipelineRun  PipelineRun "timeout-pipelinerun" was timed out but had errors trying to time out TaskRuns and/or Runs: failed to patch TaskRun `timeout-pipelinerun-fast-task` with timeout: admission webhook "validation.webhook.pipeline.tekton.dev" denied the request: validation failed: invalid value: Once the TaskRun is complete, no updates are allowed: spec

Steps to Reproduce the Problem

See the following gist for an easy way to reproduce: https://gist.github.com/hrivera-ntap/a9dc47e7924feb6dc3ad32ac13751ffb

  1. Create a pipelinerun with multiple tasks and provide a timeout to the pipelinerun
  2. PipelineRun will hit PipelineRunCouldntTimeout state

Additional Info

  • Kubernetes version:

    Output of kubectl version:

(paste your output here)
  • Tekton Pipeline version:

v0.63.0
v0.62.x

(appears to be a regression introduced after v0.61.1)

@l-qing : This appears to be closely related to the issue reported in #8172 that you recently fixed by #8173. Mind taking a look?

@hrivera-ntap hrivera-ntap added the kind/bug Categorizes issue or PR as related to a bug. label Sep 2, 2024
@l-qing
Copy link
Contributor

l-qing commented Sep 2, 2024

Thank you for your feedback. I looked at it today.

l-qing added a commit to l-qing/pipeline that referenced this issue Sep 3, 2024
fix tektoncd#8230

When the PipelineRun timeout, validation errors returned when patch a
completed TaskRun should be ignored.
l-qing added a commit to l-qing/pipeline that referenced this issue Sep 3, 2024
…ssfully

fix tektoncd#8230

When the PipelineRun timeout, validation errors returned when patch a
completed TaskRun should be ignored.
l-qing added a commit to l-qing/pipeline that referenced this issue Sep 5, 2024
…ssfully

fix tektoncd#8230

When the PipelineRun timeout, validation errors returned when patch a
completed TaskRun should be ignored.
l-qing added a commit to l-qing/pipeline that referenced this issue Sep 6, 2024
…ssfully

fix tektoncd#8230

When the PipelineRun timeout, validation errors returned when patch a
completed TaskRun should be ignored.
l-qing added a commit to l-qing/pipeline that referenced this issue Sep 13, 2024
…ssfully

fix tektoncd#8230

When the PipelineRun timeout, validation errors returned when patch a
completed TaskRun should be ignored.
l-qing added a commit to l-qing/pipeline that referenced this issue Sep 17, 2024
…ssfully

fix tektoncd#8230

When the PipelineRun timeout, validation errors returned when patch a
completed TaskRun should be ignored.
l-qing added a commit to l-qing/pipeline that referenced this issue Sep 19, 2024
…ssfully

fix tektoncd#8230

When the PipelineRun timeout, validation errors returned when patch a
completed TaskRun should be ignored.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants