Skip to content

Commit

Permalink
Make EC verify task timeout configurable
Browse files Browse the repository at this point in the history
Before this change, users could use the TIMEOUT param to increase their
timeout up from the 5m default, but they would hit a ceiling. If they
increased their timeout past 2h, then a tekton default on timeout for
the task itself would kick in.

This change will cause the timeout param to be used both for the value
passed to ec running in side the task, and for the timeout of the task
itself. When the time has expired, tekton will cancel the task.
  • Loading branch information
ralphbean committed Nov 15, 2024
1 parent 89fdc4c commit de5ee9a
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions pipelines/enterprise-contract.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,7 @@ spec:
value: "$(tasks.verify.results.TEST_OUTPUT)"
tasks:
- name: verify
timeout: "$(params.TIMEOUT)"
params:
- name: POLICY_CONFIGURATION
value: "$(params.POLICY_CONFIGURATION)"
Expand Down

0 comments on commit de5ee9a

Please sign in to comment.