Skip to content

github-pull-request-make: set minimum testrace duration per test#46138

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
pbardea:ci-timeout
Mar 16, 2020
Merged

github-pull-request-make: set minimum testrace duration per test#46138
craig[bot] merged 1 commit intocockroachdb:masterfrom
pbardea:ci-timeout

Conversation

@pbardea
Copy link
Copy Markdown
Contributor

@pbardea pbardea commented Mar 16, 2020

This change updates the minimum allowed duration for a test run under
race in CI to be configured on a per-test basis, rather than a
per-package basis.

When there are a lot of tests in a given package that need to be
stressed by testrace, the entire package may not be given sufficient
time to complete. For example, #45962 was not passing CI since many
tests were changed within one package.

Release justification: Non-production code changes.
Release note: None

@pbardea pbardea requested a review from tbg March 16, 2020 03:58
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

This change updates the minimum allowed duration for a test run under
race in CI to be configured on a per-test basis, rather than a
per-package basis.

When there are a lot of tests in a given package that need to be
stressed by testrace, the entire package may not be given sufficient
time to complete. For example, cockroachdb#45962 was not passing CI since many
tests were changed within one package.

Release justification: Non-production code changes.
Release note: None
@pbardea pbardea changed the title github-pull-request-make: allow 1 minute per testrace test github-pull-request-make: set minimum testrace duration per test Mar 16, 2020
@pbardea
Copy link
Copy Markdown
Contributor Author

pbardea commented Mar 16, 2020

TFTR!
bors r=tbg

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Mar 16, 2020

Build succeeded

@craig craig bot merged commit a06be62 into cockroachdb:master Mar 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants