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

Fix flaky pit/scroll tests in SegmentReplicationIT #10770

Merged
merged 1 commit into from
Nov 6, 2023

Conversation

mch2
Copy link
Member

@mch2 mch2 commented Oct 20, 2023

Description

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

Related Issues

Resolves #10769

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)
  • Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 20, 2023

Compatibility status:

Checks if related components are compatible with change 5e32d8b

Incompatible components

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

Skipped components

Compatible components

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

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

Copy link
Member

@dreamer-89 dreamer-89 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this.

@dreamer-89 dreamer-89 added the backport 2.x Backport to 2.x branch label Nov 3, 2023
@dreamer-89
Copy link
Member

dreamer-89 commented Nov 3, 2023

Gradle Check (Jenkins) Run Completed with:

#7934
#11079

Copy link
Contributor

github-actions bot commented Nov 3, 2023

Gradle Check (Jenkins) Run Completed with:

@dreamer-89
Copy link
Member

@mch2 : There are bunch of flaky tests failing in last few gradle checks. Can you rebase your branch with latest main?

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>
@mch2 mch2 requested a review from adnapibar as a code owner November 5, 2023 22:03
Copy link
Contributor

github-actions bot commented Nov 5, 2023

Gradle Check (Jenkins) Run Completed with:

Copy link
Contributor

github-actions bot commented Nov 5, 2023

Gradle Check (Jenkins) Run Completed with:

Copy link
Contributor

github-actions bot commented Nov 6, 2023

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.repositories.azure.AzureBlobContainerRetriesTests.testWriteLargeBlob

Copy link

codecov bot commented Nov 6, 2023

Codecov Report

Merging #10770 (5e32d8b) into main (747f7d1) will increase coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main   #10770      +/-   ##
============================================
+ Coverage     71.20%   71.25%   +0.04%     
- Complexity    58716    58774      +58     
============================================
  Files          4872     4872              
  Lines        276773   276773              
  Branches      40240    40240              
============================================
+ Hits         197078   197212     +134     
+ Misses        63238    63137     -101     
+ Partials      16457    16424      -33     

see 450 files with indirect coverage changes

@mch2
Copy link
Member Author

mch2 commented Nov 6, 2023

Gradle Check (Jenkins) Run Completed with:

#10006

@mch2
Copy link
Member Author

mch2 commented Nov 6, 2023

Gradle Check (Jenkins) Run Completed with:

#11074

@mch2 mch2 merged commit c1962cc into opensearch-project:main Nov 6, 2023
46 of 53 checks passed
@mch2 mch2 deleted the pitscrollflaky branch November 6, 2023 04:47
opensearch-trigger-bot bot pushed a commit that referenced this pull request Nov 6, 2023
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>
mch2 pushed a commit that referenced this pull request Nov 6, 2023
This change adds an assertBusy to wait until files are cleared from disk as it is not
synchronous with the scroll/pit removal.


(cherry picked from commit c1962cc)

Signed-off-by: Marc Handalian <handalm@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
rayshrey pushed a commit to rayshrey/OpenSearch that referenced this pull request Mar 18, 2024
…t#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>
shiv0408 pushed a commit to Gaurav614/OpenSearch that referenced this pull request Apr 25, 2024
…t#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>
Signed-off-by: Shivansh Arora <hishiv@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x Backport to 2.x branch bug Something isn't working skip-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] SegmentReplicationIT.testScrollCreatedOnReplica is flaky
2 participants