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

CQ: Fix flakes in the store file scan test #12721

Merged
merged 1 commit into from
Nov 14, 2024
Merged

Conversation

lhoguin
Copy link
Contributor

@lhoguin lhoguin commented Nov 14, 2024

We don't expect random bytes to be there in the current version of the message store as we overwrite empty spaces with zeroes when moving messages around.

This fixes a very rare flake.

We don't expect random bytes to be there in the current
version of the message store as we overwrite empty spaces
with zeroes when moving messages around.

We also don't expect messages to be false flagged when
the broker is running because it checks for message
validity in the index. Therefore make sure message bodies
in the tests don't contain byte 255.
@lhoguin lhoguin changed the title CQ: Change the scan test to pad with zeroes instead of random bytes CQ: Fix flakes in the store file scan test Nov 14, 2024
@michaelklishin michaelklishin merged commit 4ebfd7f into main Nov 14, 2024
273 checks passed
@michaelklishin michaelklishin deleted the loic-fix-cq-flake branch November 14, 2024 18:15
michaelklishin added a commit that referenced this pull request Nov 14, 2024
CQ: Fix flakes in the store file scan test (backport #12721)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants