makefile: increase test timeout #40838
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This patch increases the test timeout for every package from 12m to 20m (and under stress from 25m to 30m). This is motivated by the sql TestLogic which has been inching towards the 12m recently (according to TeamCity history) and is sometimes timing out. Hopefully it's because we've been adding more tests... Fixes cockroachdb#40572 Release note: None Release justification: fix timeout flakes
|
This change is |
jordanlewis
approved these changes
Sep 17, 2019
There was a problem hiding this comment.
Reviewable status:
complete! 1 of 0 LGTMs obtained (waiting on @jordanlewis)
|
bors r+
…On Tue, Sep 17, 2019 at 2:05 PM Jordan Lewis ***@***.***> wrote:
***@***.**** approved this pull request.
[image: <img class="emoji" title=":lgtm:" alt=":lgtm:" align="absmiddle" src="https://reviewable.io/lgtm.png" height="20" width="61"/>]
<https://camo.githubusercontent.com/41b3ec3419116e3b3f84507ad965646ce4ce1f0c/68747470733a2f2f72657669657761626c652e696f2f6c67746d2e706e67>
*Reviewable
<https://reviewable.io/reviews/cockroachdb/cockroach/40838#-:-Lp-OP533ms4BjlPhQ_h:bnfp4nl>*
status: [image:
] complete! 1 of 0 LGTMs obtained (waiting on
@jordanlewis <https://github.com/jordanlewis>)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#40838?email_source=notifications&email_token=AAC4C4ITNHXN5GCNOUAE3BTQKEL5DA5CNFSM4IXS5ZA2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCFAKCTQ#pullrequestreview-289448270>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAC4C4LHU6EOZWZI35EMAGDQKEL5DANCNFSM4IXS5ZAQ>
.
|
Build failed (retrying...) |
craig bot
pushed a commit
that referenced
this pull request
Sep 17, 2019
40676: sql: stop presplitting spans on every import proc r=pbardea a=pbardea We are currently presplitting spans (on every table and index boundary) for every processor. We should only be doing this once so this PR moves it to the planning stage. Addresses #39072. Release note: None 40838: makefile: increase test timeout r=andreimatei a=andreimatei This patch increases the test timeout for every package from 12m to 20m (and under stress from 25m to 30m). This is motivated by the sql TestLogic which has been inching towards the 12m recently (according to TeamCity history) and is sometimes timing out. Hopefully it's because we've been adding more tests... Fixes #40572 Release note: None Release justification: fix timeout flakes Co-authored-by: Paul Bardea <pbardea@gmail.com> Co-authored-by: Andrei Matei <andrei@cockroachlabs.com>
Build succeeded |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This patch increases the test timeout for every package from 12m to 20m
(and under stress from 25m to 30m).
This is motivated by the sql TestLogic which has been inching towards
the 12m recently (according to TeamCity history) and is sometimes timing
out. Hopefully it's because we've been adding more tests...
Fixes #40572
Release note: None
Release justification: fix timeout flakes