-
Notifications
You must be signed in to change notification settings - Fork 5
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
chore(apps/prod/tekton/configs/triggers): increase pipeline run timeout to 3h #1183
Conversation
…ut to 3h avoid timeout on low performance mac machines. Signed-off-by: wuhuizuo <wuhuizuo@126.com>
The pull request increases the pipeline run timeout from 2 hours to 3 hours for the triggers in There are no potential problems with this change. However, it is important to note that increasing the timeout may result in longer wait times for builds to complete. One suggestion to improve the pull request is to provide more context. It would be helpful to know how the low-performance Mac machines were affecting the pipeline and why the timeout needed to be increased. Another suggestion is to consider alternative solutions to the timeout problem, such as optimizing the pipeline or upgrading the machine's performance. Overall, this pull request seems reasonable and can be merged. |
PR-Agent was enabled for this repository. To continue using it, please link your git user with your CodiumAI identity here. PR Reviewer Guide 🔍
|
/approve |
PR-Agent was enabled for this repository. To continue using it, please link your git user with your CodiumAI identity here. PR Code Suggestions ✨
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: purelind, wuhuizuo The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
[LGTM Timeline notifier]Timeline:
|
User description
avoid timeout on low performance mac machines.
Signed-off-by: wuhuizuo wuhuizuo@126.com
PR Type
Enhancement
Description
tiflash
andtikv
components to avoid timeouts on low-performance Mac machines.Changes walkthrough 📝
git-push.yaml
Increase pipeline run timeout for tiflash to 3 hours
apps/prod/tekton/configs/triggers/triggers/pingcap/tiflash/git-push.yaml
git-push.yaml
Increase pipeline run timeout for tikv to 3 hours
apps/prod/tekton/configs/triggers/triggers/tikv/tikv/git-push.yaml