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

[Backport 2.x] Fix flaky pit/scroll tests in SegmentReplicationIT #11098

Merged
merged 1 commit into from
Nov 6, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport c1962cc from #10770.

This change adds an assertBusy to wait until files are cleared from disk as it is not
synchronous with the scroll/pit removal.

Signed-off-by: Marc Handalian <handalm@amazon.com>
(cherry picked from commit c1962cc)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Copy link
Contributor

github-actions bot commented Nov 6, 2023

Compatibility status:

Checks if related components are compatible with change 76f3ed9

Incompatible components

Incompatible components: [https://github.com/opensearch-project/performance-analyzer.git]

Skipped components

Compatible components

Compatible components: [https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/alerting.git]

Copy link
Contributor

github-actions bot commented Nov 6, 2023

Gradle Check (Jenkins) Run Completed with:

Copy link
Contributor

github-actions bot commented Nov 6, 2023

Gradle Check (Jenkins) Run Completed with:

@mch2
Copy link
Member

mch2 commented Nov 6, 2023

Gradle Check (Jenkins) Run Completed with:

#10749
#10798
#9332

@mch2
Copy link
Member

mch2 commented Nov 6, 2023

Gradle Check (Jenkins) Run Completed with:

#11109
#10046

Copy link
Contributor

github-actions bot commented Nov 6, 2023

Gradle Check (Jenkins) Run Completed with:

@mch2
Copy link
Member

mch2 commented Nov 6, 2023

Gradle Check (Jenkins) Run Completed with:

#11074 again

Copy link
Contributor

github-actions bot commented Nov 6, 2023

Gradle Check (Jenkins) Run Completed with:

Copy link

codecov bot commented Nov 6, 2023

Codecov Report

Merging #11098 (76f3ed9) into 2.x (7bf2a3a) will decrease coverage by 0.08%.
Report is 1 commits behind head on 2.x.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                2.x   #11098      +/-   ##
============================================
- Coverage     70.98%   70.90%   -0.08%     
+ Complexity    58932    58892      -40     
============================================
  Files          4851     4851              
  Lines        278030   278030              
  Branches      40766    40766              
============================================
- Hits         197357   197140     -217     
- Misses        63916    64113     +197     
- Partials      16757    16777      +20     

see 478 files with indirect coverage changes

@mch2 mch2 merged commit 05c2bef into 2.x Nov 6, 2023
42 of 67 checks passed
@mch2 mch2 deleted the backport/backport-10770-to-2.x branch November 6, 2023 20:28
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