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

Ensure that segments are upload to remote store in case of local and … #10948

Merged
merged 4 commits into from
Oct 28, 2023

Conversation

gbbafna
Copy link
Collaborator

@gbbafna gbbafna commented Oct 26, 2023

…snapshot based recovery

Description

Ensure that segments are upload to remote store in case of local and snapshot recovery before activating the primary mode.

Related Issues

Resolves #10957

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
    - [x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
    - [x] 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.

…snapshot based recovery

Signed-off-by: Gaurav Bafna <gbbafna@amazon.com>
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Oct 26, 2023

Compatibility status:

Checks if related components are compatible with change b987c55

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/geospatial.git, https://github.com/opensearch-project/ml-commons.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:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      2 org.opensearch.index.shard.RemoteIndexShardTests.testNoFailuresOnFileReads

@codecov
Copy link

codecov bot commented Oct 26, 2023

Codecov Report

Merging #10948 (0fac132) into main (fe8b2d5) will increase coverage by 0.08%.
Report is 6 commits behind head on main.
The diff coverage is 4.00%.

❗ Current head 0fac132 differs from pull request most recent head b987c55. Consider uploading reports for the commit b987c55 to get more accurate results

@@             Coverage Diff              @@
##               main   #10948      +/-   ##
============================================
+ Coverage     71.11%   71.20%   +0.08%     
- Complexity    58687    58709      +22     
============================================
  Files          4872     4872              
  Lines        276691   276715      +24     
  Branches      40221    40231      +10     
============================================
+ Hits         196770   197029     +259     
+ Misses        63523    63241     -282     
- Partials      16398    16445      +47     
Files Coverage Δ
...search/index/shard/RemoteStoreRefreshListener.java 85.09% <20.00%> (-1.67%) ⬇️
...in/java/org/opensearch/index/shard/IndexShard.java 69.77% <0.00%> (+0.48%) ⬆️
...java/org/opensearch/index/shard/StoreRecovery.java 55.21% <0.00%> (-2.01%) ⬇️

... and 451 files with indirect coverage changes

Copy link
Member

@ashking94 ashking94 left a comment

Choose a reason for hiding this comment

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

LGTM!!

@ashking94 ashking94 added the backport 2.x Backport to 2.x branch label Oct 27, 2023
@ashking94
Copy link
Member

Lets add tests as well. Ty!

Signed-off-by: Gaurav Bafna <gbbafna@amazon.com>
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

Signed-off-by: Gaurav Bafna <gbbafna@amazon.com>
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

Signed-off-by: Gaurav Bafna <gbbafna@amazon.com>
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@sachinpkale
Copy link
Member

Flaky tests are failing.

@sachinpkale sachinpkale merged commit 45f7be1 into opensearch-project:main Oct 28, 2023
14 of 15 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Oct 28, 2023
#10948)

---------

Signed-off-by: Gaurav Bafna <gbbafna@amazon.com>
(cherry picked from commit 45f7be1)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
gbbafna pushed a commit that referenced this pull request Oct 30, 2023
#10948) (#10974)

---------


(cherry picked from commit 45f7be1)

Signed-off-by: Gaurav Bafna <gbbafna@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>
shiv0408 pushed a commit to Gaurav614/OpenSearch that referenced this pull request Apr 25, 2024
opensearch-project#10948)

---------

Signed-off-by: Gaurav Bafna <gbbafna@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.

[Remote Store] Durability issue with indices created via split/shrink/clone/manual snapshot recovery
4 participants