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] Validate checksum of each segment file post download from remote store #10590

Merged
merged 1 commit into from
Oct 13, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 6c02261 from #10119.

#10119)

---------

Signed-off-by: Sachin Kale <[email protected]>
Co-authored-by: Sachin Kale <[email protected]>
(cherry picked from commit 6c02261)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Contributor

Compatibility status:

Checks if related components are compatible with change a1533b5

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/job-scheduler.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/observability.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/performance-analyzer.git, https://github.com/opensearch-project/ml-commons.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

Gradle Check (Jenkins) Run Completed with:

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

@codecov
Copy link

codecov bot commented Oct 12, 2023

Codecov Report

Merging #10590 (a1533b5) into 2.x (52018ec) will increase coverage by 0.03%.
The diff coverage is 76.00%.

@@             Coverage Diff              @@
##                2.x   #10590      +/-   ##
============================================
+ Coverage     70.84%   70.87%   +0.03%     
- Complexity    58489    58536      +47     
============================================
  Files          4831     4831              
  Lines        276413   276436      +23     
  Branches      40581    40583       +2     
============================================
+ Hits         195822   195929     +107     
+ Misses        63872    63829      -43     
+ Partials      16719    16678      -41     
Files Coverage Δ
...earch/index/store/RemoteSegmentStoreDirectory.java 88.88% <100.00%> (+0.03%) ⬆️
...rc/main/java/org/opensearch/index/store/Store.java 80.30% <75.00%> (-1.24%) ⬇️

... and 478 files with indirect coverage changes

@gbbafna gbbafna merged commit 63d531c into 2.x Oct 13, 2023
@github-actions github-actions bot deleted the backport/backport-10119-to-2.x branch October 13, 2023 10:17
sachinpkale pushed a commit to sachinpkale/OpenSearch that referenced this pull request Oct 24, 2023
kotwanikunal pushed a commit that referenced this pull request Oct 25, 2023
…ote store (#10119) (#10590)" (#10878)

This reverts commit 63d531c.

Signed-off-by: Sachin Kale <[email protected]>
Co-authored-by: Sachin Kale <[email protected]>
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