Skip to content

roachtest: reenable Jepsen subcritical-skews test#112710

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
renatolabs:rc/reenable-jepsen-subcritial-skews
Oct 20, 2023
Merged

roachtest: reenable Jepsen subcritical-skews test#112710
craig[bot] merged 1 commit intocockroachdb:masterfrom
renatolabs:rc/reenable-jepsen-subcritial-skews

Conversation

@renatolabs
Copy link
Copy Markdown

The Jepsen version we are using already moved from ntp.ubuntu.org to pool.ntp.org. We should be able to run these tests again.

https://github.com/cockroachdb/jepsen/blob/cdeef40a0cd24af0c989e0a7990ee1c7fa948f43/cockroachdb/src/jepsen/cockroach/time.clj#L27

Fixes: #35599

Release note: None

@renatolabs renatolabs requested a review from a team as a code owner October 19, 2023 19:46
@renatolabs renatolabs requested review from DarrylWong and herkolategan and removed request for a team October 19, 2023 19:46
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

The Jepsen version we are using already moved from `ntp.ubuntu.org` to
`pool.ntp.org`. We should be able to run these tests again.

https://github.com/cockroachdb/jepsen/blob/cdeef40a0cd24af0c989e0a7990ee1c7fa948f43/cockroachdb/src/jepsen/cockroach/time.clj#L27

Fixes: cockroachdb#35599

Release note: None
@renatolabs renatolabs force-pushed the rc/reenable-jepsen-subcritial-skews branch from 6fa42dc to 32f7a71 Compare October 19, 2023 20:24
@renatolabs
Copy link
Copy Markdown
Author

Subcritical-skews tests passed: https://teamcity.cockroachdb.com/viewLog.html?buildId=12260161&buildTypeId=Cockroach_Nightlies_RoachtestNightlyGceBazel&tab=testsInfo&branch_Cockroach_Nightlies=112710.

Still remains to be seen whether we'll get rate limited over time, but I think it's worth re-enabling these tests and seeing what happens.

@renatolabs renatolabs added backport-23.1.x PAST MAINTENANCE SUPPORT: 23.1 patch releases via ER request only backport-23.2.x PAST MAINTENANCE SUPPORT: 23.2 patch releases via ER request only labels Oct 20, 2023
@renatolabs
Copy link
Copy Markdown
Author

TFTR!

bors r=DarrylWong

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Oct 20, 2023

Build succeeded:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-23.1.x PAST MAINTENANCE SUPPORT: 23.1 patch releases via ER request only backport-23.2.x PAST MAINTENANCE SUPPORT: 23.2 patch releases via ER request only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

roachtest: jepsen subcritical-skews tests a skipped due to ntp rate limiting

3 participants