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.

Sign up for GitHub

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

Jump to bottom

issue-122: AddConfirmedBlobs should do ProcessCommitQueue, otherwise compaction may get stuck #863

Merged
merged 2 commits into from
Mar 30, 2024
Merged

issue-122: AddConfirmedBlobs should do ProcessCommitQueue, otherwise compaction may get stuck #863

merged 2 commits into from
Mar 30, 2024

Conversation

SvartMetal
Copy link
Collaborator

@SvartMetal SvartMetal commented Mar 29, 2024 •
edited
Loading

No description provided.

@SvartMetal SvartMetal force-pushed the users/svartmetal/issue-122/70 branch from 6433a8d to 2d1ff63 Compare March 29, 2024 23:51
@SvartMetal SvartMetal changed the title issue-122: AddConfirmedBlobs should do ProcessCommitQueue issue-122: AddConfirmedBlobs should do ProcessCommitQueue, otherwise compaction may get stuck Mar 29, 2024
@github-actions GitHub Actions
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 2d1ff63.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5075 5072 0 0 1 2

@github-actions GitHub Actions
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit 38c7ccf.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5079 5073 0 6 0 0

@SvartMetal
Copy link
Collaborator Author 

cloud/blockstore/tests/plugin/uds/test.py.test_load[stable-endpoint2] | 10m 24.817s | FAIL(TIMEOUT)
cloud/blockstore/tests/plugin/uds/test.py.test_load[trunk-endpoint2] | 10m 24.444s | FAIL(TIMEOUT)
cloud/blockstore/tests/plugin/uds/test.py.test_load[trunk-endpoint_nbd] | 10m 29.141s | FAIL(TIMEOUT)

Tests are flapping

@SvartMetal SvartMetal merged commit b1d617a into main Mar 30, 2024
6 of 7 checks passed
@SvartMetal SvartMetal deleted the users/svartmetal/issue-122/70 branch March 30, 2024 14:36
SvartMetal added a commit that referenced this pull request Mar 30, 2024
@SvartMetal
issue-122: AddConfirmedBlobs should do ProcessCommitQueue, otherwise …
a9d7f7d 
…compaction may get stuck (#863)
SvartMetal added a commit that referenced this pull request Mar 31, 2024
@SvartMetal
Merge to stable-23-3 (#864)
38b655c 
* [Blockstore] output WriteAndZeroRequestsInProgress on partition mon-page (#847)

* [Blockstore] verify that GroupRequests does not forget requests (#846)

* issue-122: AddConfirmedBlobs should do ProcessCommitQueue, otherwise compaction may get stuck (#863)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@drbasic drbasic drbasic approved these changes

@qkrorlqr qkrorlqr qkrorlqr approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@SvartMetal @qkrorlqr @drbasic