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

tests: Raise the rounding timestamp error threshold for testing #12916

Merged
merged 2 commits into from
Aug 23, 2023

Conversation

graphcareful
Copy link
Contributor

There have been some reports of failures in a usage test that is due to a timer being fired a second later then normal. Given some possible discrepancy with rounding and that the error has only been observed in cases where the observed timestamp is 1s over the expected, this patch raises the error threshold for the timer to two seconds.

Fixes: #12678

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v23.2.x
  • v23.1.x
  • v22.3.x

Release Notes

  • none

@graphcareful
Copy link
Contributor Author

@graphcareful graphcareful merged commit 3e31dad into redpanda-data:dev Aug 23, 2023
26 checks passed
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.

CI Failure (AssertionError when there are no gaps between windows) in UsageTest.test_usage_metrics_collection
2 participants