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

[Remote Translog] Fix translog inconsistent generation assertion failure in ITs #10985

Merged
merged 1 commit into from
Nov 12, 2023

Conversation

ashking94
Copy link
Member

@ashking94 ashking94 commented Oct 30, 2023

Description

As seen in #10902, we can see that during test cleanup, we see an assertion failure that happens due to max reader generation being different than supplied generation. This will fix multiple flaky tests that fail due to this reason.

Related Issues

Resolves #10902

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
  • 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.

…ure in ITs

Signed-off-by: Ashish Singh <ssashish@amazon.com>
@github-actions github-actions bot added :test Adding or fixing a test bug Something isn't working flaky-test Random test failure that succeeds on second run labels Oct 30, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Oct 30, 2023

Compatibility status:

Checks if related components are compatible with change 06df33a

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/sql.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/custom-codecs.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/opensearch-oci-object-storage.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:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.remotestore.SegmentReplicationUsingRemoteStoreIT.testIndexReopenClose

@ashking94
Copy link
Member Author

org.opensearch.remotestore.SegmentReplicationUsingRemoteStoreIT.testIndexReopenClose

Created issue for test failure as it does not seem to be related to the code change done here - #10987

Copy link
Contributor

github-actions bot commented Nov 3, 2023

Gradle Check (Jenkins) Run Completed with:

Copy link

codecov bot commented Nov 3, 2023

Codecov Report

Merging #10985 (06df33a) into main (73bbeb5) will decrease coverage by 0.03%.
Report is 18 commits behind head on main.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main   #10985      +/-   ##
============================================
- Coverage     71.27%   71.25%   -0.03%     
- Complexity    58724    58747      +23     
============================================
  Files          4870     4870              
  Lines        276578   276578              
  Branches      40202    40202              
============================================
- Hits         197137   197069      -68     
- Misses        62986    63073      +87     
+ Partials      16455    16436      -19     
Files Coverage Δ
...rg/opensearch/index/translog/RemoteFsTranslog.java 75.60% <100.00%> (ø)

... and 493 files with indirect coverage changes

@mch2 mch2 merged commit b01e483 into opensearch-project:main Nov 12, 2023
118 of 153 checks passed
@mch2 mch2 added the backport 2.x Backport to 2.x branch label Nov 12, 2023
opensearch-trigger-bot bot pushed a commit that referenced this pull request Nov 12, 2023
…ure in ITs (#10985)

(cherry picked from commit b01e483)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
mch2 pushed a commit that referenced this pull request Nov 17, 2023
…ure in ITs (#10985) (#11168)

(cherry picked from commit b01e483)
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>
fahadshamiinsta pushed a commit to fahadshamiinsta/OpenSearch270 that referenced this pull request Dec 4, 2023
rayshrey pushed a commit to rayshrey/OpenSearch that referenced this pull request Mar 18, 2024
shiv0408 pushed a commit to Gaurav614/OpenSearch that referenced this pull request Apr 25, 2024
…ure in ITs (opensearch-project#10985)

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 flaky-test Random test failure that succeeds on second run skip-changelog :test Adding or fixing a test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Test SegmentReplicationUsingRemoteStoreDisruptionIT.testCancelReplicationWhileFetchingMetadata is flaky
3 participants