making TIndexTabletTest_Data::ShouldReturnBackendInfoForIOForOverloadedTabletActor more stable#6591
making TIndexTabletTest_Data::ShouldReturnBackendInfoForIOForOverloadedTabletActor more stable#6591qkrorlqr wants to merge 1 commit into
Conversation
|
Note This is an automated comment that will be appended during run. Note All workloads for linux-x86_64-relwithdebinfo have completed. Tip Planned checks for linux-x86_64-relwithdebinfo. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 080dcc5ab1
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
…edTabletActor more stable
080dcc5 to
436f56f
Compare
|
Note This is an automated comment that will be appended during run. Note All workloads for linux-x86_64-relwithdebinfo have completed. Tip Planned checks for linux-x86_64-relwithdebinfo. 🟢 linux-x86_64-relwithdebinfo target: cloud/filestore/ (test time: 5188s): all tests PASSED for commit 436f56f.
|
Notes
Decreasing the probability of things like this: https://github-actions-s3.storage.eu-north2.nebius.cloud/ydb-platform/nbs/Nightly-build-(tsan)/29879592736/1/nebius-x86-64-tsan/logs/1/cloud/filestore/libs/storage/tablet/ut/test-results/unittest/chunk3/testing_out_stuff/TIndexTabletTest_Data.ShouldReturnBackendInfoForIOForOverloadedTabletActor128_KB.err
The test used to spend wall time doing read/write-blob ops caused by background ops which could sometimes cause the tablet to just wait without doing anything and without recording cpu cycles so eventually we could end up not recording any overload because of the following event sequence:
Issue
None