Skip to content

Add Snapshot and Remote Restore ITs for Warm Index #18281

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

Merged
merged 17 commits into from
May 27, 2025

Conversation

Gagan6164
Copy link
Contributor

Description

Making the Snapshot IT's to also run with warm index

Related Issues

Resolves #[Issue number to be closed when this PR is merged]

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

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.

Signed-off-by: Gagan Singh Saini <[email protected]>
Copy link
Contributor

❌ Gradle check result for ae4d9d5: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Copy link
Contributor

❌ Gradle check result for 2f3c89d: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Signed-off-by: Gagan Singh Saini <[email protected]>
Copy link
Contributor

❌ Gradle check result for f8e8a37: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Copy link
Contributor

❕ Gradle check result for b9b5359: UNSTABLE

Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

Copy link

codecov bot commented May 15, 2025

Codecov Report

Attention: Patch coverage is 80.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 71.87%. Comparing base (58c281f) to head (abb55c1).
Report is 49 commits behind head on main.

Files with missing lines Patch % Lines
...org/opensearch/index/store/CompositeDirectory.java 85.71% 2 Missing ⚠️
...in/java/org/opensearch/index/shard/IndexShard.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main   #18281      +/-   ##
============================================
- Coverage     72.44%   71.87%   -0.57%     
+ Complexity    67256    66653     -603     
============================================
  Files          5488     5488              
  Lines        310994   311034      +40     
  Branches      45212    45218       +6     
============================================
- Hits         225288   223566    -1722     
- Misses        67297    68979    +1682     
- Partials      18409    18489      +80     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: Gagan6164 <[email protected]>
Copy link
Contributor

✅ Gradle check result for d759388: SUCCESS

Copy link
Contributor

✅ Gradle check result for cf2c194: SUCCESS

@gbbafna gbbafna changed the title Snapshot IT's with Warm Index setup Add Snapshot and Remote Restore ITs for Warm Index May 26, 2025
Signed-off-by: Gagan Singh Saini <[email protected]>
@Gagan6164 Gagan6164 force-pushed the warmIndexSnapshotIT branch from 308736c to ee8a9ba Compare May 26, 2025 08:53
Copy link
Contributor

❌ Gradle check result for ee8a9ba:

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Signed-off-by: Gagan Singh Saini <[email protected]>
Signed-off-by: Gagan Singh Saini <[email protected]>
Copy link
Contributor

❌ Gradle check result for 4124fbd: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Signed-off-by: Gagan Singh Saini <[email protected]>
Copy link
Contributor

❌ Gradle check result for 603e5d9: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Gagan Singh Saini added 2 commits May 27, 2025 11:05
Signed-off-by: Gagan Singh Saini <[email protected]>
Copy link
Contributor

❕ Gradle check result for abb55c1: UNSTABLE

Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

@gbbafna gbbafna merged commit 3b018ad into opensearch-project:main May 27, 2025
33 of 34 checks passed
Gagan6164 added a commit to Gagan6164/OpenSearch that referenced this pull request Jun 8, 2025
Gagan6164 added a commit to Gagan6164/OpenSearch that referenced this pull request Jun 8, 2025
neuenfeldttj pushed a commit to neuenfeldttj/OpenSearch that referenced this pull request Jun 26, 2025
neuenfeldttj pushed a commit to neuenfeldttj/OpenSearch that referenced this pull request Jun 26, 2025
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.

4 participants