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 #10986

Closed
wants to merge 1 commit into from

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.

@ashking94 ashking94 self-assigned this Oct 30, 2023
@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

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Oct 30, 2023

Compatibility status:

Checks if related components are compatible with change 6e527d9

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/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/opensearch-oci-object-storage.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:

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

@codecov
Copy link

codecov bot commented Oct 30, 2023

Codecov Report

Merging #10986 (6e527d9) into main (63aff16) will increase coverage by 0.00%.
Report is 5 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##               main   #10986   +/-   ##
=========================================
  Coverage     71.29%   71.29%           
- Complexity    58742    58777   +35     
=========================================
  Files          4872     4872           
  Lines        276777   276781    +4     
  Branches      40240    40242    +2     
=========================================
+ Hits         197316   197341   +25     
- Misses        62943    62998   +55     
+ Partials      16518    16442   -76     
Files Coverage Δ
...rg/opensearch/index/translog/RemoteFsTranslog.java 75.47% <100.00%> (-0.84%) ⬇️
...s/replication/SegmentReplicationTargetService.java 56.17% <100.00%> (+0.56%) ⬆️

... and 483 files with indirect coverage changes

Copy link
Contributor

github-actions bot commented Nov 1, 2023

Gradle Check (Jenkins) Run Completed with:

@ashking94
Copy link
Member Author

Gradle Check (Jenkins) Run Completed with:

flaky tests - #10006, #10755

…ure in ITs

Signed-off-by: Ashish Singh <ssashish@amazon.com>
Copy link
Contributor

github-actions bot commented Nov 1, 2023

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.remotestore.multipart.RemoteStoreMultipartIT.testAsyncDurabilityWhenRestrictSettingImplicitFalse

@ashking94
Copy link
Member Author

org.opensearch.remotestore.multipart.RemoteStoreMultipartIT.testAsyncDurabilityWhenRestrictSettingImplicitFalse

Created #11051 for the flaky test

@ashking94
Copy link
Member Author

Closing this in interest of alternate approach -> #10985.

@ashking94 ashking94 closed this Nov 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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
2 participants