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 bugs causing red indexes with remote indexes during translog upload & store recovery #10497

Merged
merged 1 commit into from
Oct 9, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 8bb11a6 from #10449.

…ad & store recovery (#10449)

---------

Signed-off-by: Ashish Singh <[email protected]>
(cherry picked from commit 8bb11a6)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2023

Compatibility status:

Checks if related components are compatible with change 86b6854

Incompatible components

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/asynchronous-search.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/neural-search.git]

@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2023

Gradle Check (Jenkins) Run Completed with:

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

@codecov
Copy link

codecov bot commented Oct 7, 2023

Codecov Report

Merging #10497 (86b6854) into 2.x (42f10d5) will increase coverage by 0.11%.
Report is 3 commits behind head on 2.x.
The diff coverage is 50.00%.

@@             Coverage Diff              @@
##                2.x   #10497      +/-   ##
============================================
+ Coverage     70.82%   70.94%   +0.11%     
- Complexity    58462    58520      +58     
============================================
  Files          4829     4829              
  Lines        276380   276384       +4     
  Branches      40579    40579              
============================================
+ Hits         195747   196068     +321     
+ Misses        63970    63571     -399     
- Partials      16663    16745      +82     
Files Coverage Δ
...ibs/core/src/main/java/org/opensearch/Version.java 83.18% <100.00%> (+0.07%) ⬆️
...c/main/java/org/opensearch/index/IndexService.java 75.65% <ø> (+0.21%) ⬆️
.../org/opensearch/index/remote/RemoteStoreUtils.java 96.55% <ø> (-0.12%) ⬇️
...earch/index/store/RemoteSegmentStoreDirectory.java 88.88% <100.00%> (-0.95%) ⬇️
...ex/translog/transfer/TranslogTransferMetadata.java 85.00% <100.00%> (+0.38%) ⬆️
...in/java/org/opensearch/index/shard/IndexShard.java 69.19% <0.00%> (+0.08%) ⬆️

... and 482 files with indirect coverage changes

@ashking94
Copy link
Member

@gbbafna Good to merge this?

@gbbafna gbbafna merged commit f1b4854 into 2.x Oct 9, 2023
41 of 67 checks passed
@github-actions github-actions bot deleted the backport/backport-10449-to-2.x branch October 9, 2023 06:18
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