Skip to content

Conversation

@yupeng9
Copy link
Contributor

@yupeng9 yupeng9 commented Mar 26, 2025

Description

There's some flakiness of Kinesis integration tests, likely due to the fact that in LocalStack, the message propagation or availability might not be immediate. So when we immediately attempt to consume it, it might not be fully "visible" depending on timing, especially with a rewind to a precise AT_SEQUENCE_NUMBER. Tweaked the test with some more produced messages, and longer sleep so the records are committed and readable.

Ran the same tests 5 times locally, and test all passed.

@github-actions
Copy link
Contributor

❌ Gradle check result for 17abbaf: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Contributor

❌ Gradle check result for b60c01b:

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Contributor

❌ Gradle check result for b60c01b:

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Contributor

❌ Gradle check result for b60c01b:

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

yupeng9 added 3 commits March 28, 2025 07:40
Signed-off-by: Yupeng Fu <yupeng@uber.com>
Signed-off-by: Yupeng Fu <yupeng@uber.com>
Signed-off-by: Yupeng Fu <yupeng@uber.com>
@github-actions
Copy link
Contributor

✅ Gradle check result for 7603930: SUCCESS

@codecov
Copy link

codecov bot commented Mar 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.41%. Comparing base (399188f) to head (7603930).
Report is 2 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main   #17697      +/-   ##
============================================
- Coverage     72.46%   72.41%   -0.06%     
- Complexity    65855    65862       +7     
============================================
  Files          5322     5322              
  Lines        305477   305477              
  Branches      44312    44312              
============================================
- Hits         221359   221197     -162     
- Misses        66023    66133     +110     
- Partials      18095    18147      +52     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@msfroh msfroh merged commit edd854a into opensearch-project:main Mar 28, 2025
31 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.

2 participants