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

*: un-skip many tests that were skipped over the last month #117833

Merged
merged 1 commit into from
Jan 16, 2024

Conversation

rickystewart
Copy link
Collaborator

This partially reverts the following commits:

e615aec402e
99884f8d5ac
99f97609926
75c91c20f50
5c64a2289dd
c1778b7a258
381c96b20f5
f9b20ae3947
8608499be4a
eb71a831a57
337777f8384

These tests were skipped due to failures under RBE, typically under deadlock and race. With the addition of the heavy pool for these tests, we anticipate the decreased load will cause these tests to stop failing, therefore we un-skip the skipped tests.

Epic: CRDB-8308
Release note: None

@rickystewart rickystewart requested a review from rail January 16, 2024 20:16
@rickystewart rickystewart requested a review from a team as a code owner January 16, 2024 20:16
@rickystewart rickystewart requested a review from a team January 16, 2024 20:16
@rickystewart rickystewart requested review from a team as code owners January 16, 2024 20:16
@rickystewart rickystewart requested a review from a team January 16, 2024 20:16
@rickystewart rickystewart requested review from a team as code owners January 16, 2024 20:16
@rickystewart rickystewart requested review from dt and rharding6373 and removed request for a team January 16, 2024 20:16
@cockroach-teamcity
Copy link
Member

This change is Reviewable

This partially reverts the following commits:

```
e615aec
99884f8
99f9760
75c91c2
5c64a22
c1778b7
381c96b
f9b20ae
8608499
eb71a83
337777f
```

These tests were skipped due to failures under RBE, typically under
`deadlock` and `race`. With the addition of the `heavy` pool for these
tests, we anticipate the decreased load will cause these tests to stop
failing, therefore we un-skip the skipped tests.

Epic: CRDB-8308
Release note: None
@rickystewart
Copy link
Collaborator Author

bors r=rail

@craig
Copy link
Contributor

craig bot commented Jan 16, 2024

Build succeeded:

@craig craig bot merged commit 8daefd5 into cockroachdb:master Jan 16, 2024
8 of 9 checks passed
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