Skip to content

Conversation

@tbg
Copy link
Member

@tbg tbg commented Oct 27, 2025

The statsLoop could still be running. In the linked test flake below, we see it call (*pebble).GetMetrics when the engines are already closed.

Rearrange the code so that we wait for statsLoop to terminate properly before exiting the test.

Closes #156138.

Epic: none

The `statsLoop` could still be running. In the linked test flake below, we see
it call `(*pebble).GetMetrics` when the engines are already closed.

Rearrange the code so that we wait for `statsLoop` to terminate properly before exiting the test.

Closes cockroachdb#156138.

Epic: none
@tbg tbg requested a review from arulajmani October 27, 2025 16:04
@tbg tbg requested a review from a team as a code owner October 27, 2025 16:04
@tbg tbg added the backport-all Flags PRs that need to be backported to all supported release branches label Oct 27, 2025
@blathers-crl
Copy link

blathers-crl bot commented Oct 27, 2025

It looks like your PR touches production code but doesn't add or edit any test code. Did you consider adding tests to your PR?

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@cockroach-teamcity
Copy link
Member

This change is Reviewable

Copy link
Collaborator

@arulajmani arulajmani left a comment

Choose a reason for hiding this comment

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

:lgtm:

@arulajmani reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @tbg)

@tbg
Copy link
Member Author

tbg commented Oct 27, 2025

The self-hosted runner lost communication with the server. Verify the machine is running and has a healthy network connection. Anything in your workflow that terminates the runner process, starves it for CPU/Memory, or blocks its network access can cause this error.

bors r+

TFTR!

@craig
Copy link
Contributor

craig bot commented Oct 27, 2025

@craig craig bot merged commit 35ec42c into cockroachdb:master Oct 27, 2025
23 of 24 checks passed
@blathers-crl
Copy link

blathers-crl bot commented Oct 27, 2025

Encountered an error creating backports. Some common things that can go wrong:

  1. The backport branch might have already existed.
  2. There was a merge conflict.
  3. The backport branch contained merge commits.

You might need to create your backport manually using the backport tool.


error creating merge commit from 7880791 to blathers/backport-release-24.3-156288: POST https://api.github.com/repos/tbg/cockroach/merges: 409 Merge conflict []

you may need to manually resolve merge conflicts with the backport tool.

Backport to branch release-24.3 failed. See errors above.


🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

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

Labels

backport-all Flags PRs that need to be backported to all supported release branches backport-failed target-release-26.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

kv/kvserver/raftstorebench: TestBenchmarkRaftStoreSmokeTest failed

4 participants