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

roachtest: bump timeout for hibernate tests #114378

Merged
merged 1 commit into from
Nov 14, 2023

Conversation

rafiss
Copy link
Collaborator

@rafiss rafiss commented Nov 13, 2023

The hibernate tests are massive (11519 tests), and are slow. We are hitting against the default timeout of 3 hours, so this commit increases that to 4.

fixes #114301
fixes #114136
Release note: None

The hibernate tests are massive (11519 tests), and are slow.
We are hitting against the default timeout of 3 hours, so this commit
increases that to 4.

Release note: None
@rafiss rafiss added backport-23.1.x Flags PRs that need to be backported to 23.1 backport-23.2.x Flags PRs that need to be backported to 23.2. labels Nov 13, 2023
@rafiss rafiss requested a review from annrpom November 13, 2023 21:36
@rafiss rafiss requested review from a team as code owners November 13, 2023 21:36
@rafiss rafiss requested review from srosenberg and DarrylWong and removed request for a team November 13, 2023 21:36
@cockroach-teamcity
Copy link
Member

This change is Reviewable

Copy link
Contributor

@DarrylWong DarrylWong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry if this was mentioned or obvious, but I think important context is that these tests were both metamorphic builds, which can and probably due slow down the test slightly.

It looks like both failures almost finished though (in the results checking phase) so bumping the timeout still seems like the right thing to do.

Copy link
Contributor

@annrpom annrpom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DARRYL BEAT ME TO IT lgtm

@rafiss
Copy link
Collaborator Author

rafiss commented Nov 13, 2023

tftr!

bors r+

@craig
Copy link
Contributor

craig bot commented Nov 14, 2023

Build succeeded:

@craig craig bot merged commit 0d44995 into cockroachdb:master Nov 14, 2023
@rafiss rafiss deleted the bump-hibernate-timeout branch November 14, 2023 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-23.1.x Flags PRs that need to be backported to 23.1 backport-23.2.x Flags PRs that need to be backported to 23.2.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

roachtest: hibernate failed roachtest: hibernate failed
4 participants