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

ignoring TEvBootstrapper::TEvStatus in TVolumeActor::StateZombie (fixing rare crashes in tests) #272

Merged
merged 1 commit into from
Jan 29, 2024

Conversation

qkrorlqr
Copy link
Collaborator

No description provided.

Copy link
Contributor

Note

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

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit 26dcd6c.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
4700 4627 0 1 0 72

@qkrorlqr
Copy link
Collaborator Author

упал тест cloud/blockstore/tests/notify/test.py.test_notify[user-id]

флап

@qkrorlqr qkrorlqr merged commit 984cbf5 into main Jan 29, 2024
4 of 5 checks passed
@qkrorlqr qkrorlqr deleted the users/qkrorlqr/misc branch January 29, 2024 14:31
qkrorlqr added a commit that referenced this pull request Jan 30, 2024
qkrorlqr added a commit that referenced this pull request Jan 30, 2024
* ignoring TEvBootstrapper::TEvStatus in TVolumeActor::StateZombie (fixing rare crashes in tests) (#272)

* issue #196: implemented block checksum calculation and verification for the blobs stored in blobstorage (#278)

* issue #196: implemented calculation of block checksums in the blobs stored in blobstorage; implemented checksum checking for those blobs upon read; enabled this logic for the first 1GiB of the disk space in most of the uts in part_ut and in tests/loadtest/local large test suite; will add the uts that directly check checksum verification is the next PR;

* issue #196: review-related cleanup;

* issue #196: more review-related cleanup;

* issue #196: added a ut with CheckBlockChecksumsInBlobsUponRead, fixed a bug in block verification in ReadBlocks data path, enabled CheckBlockChecksumsInBlobsUponRead in tests/loadtest/local

* issue #196: added more uts

* fixed build after merge
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants