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] Snapshot Status API changes #15665

Merged

Conversation

ltaragi
Copy link
Contributor

@ltaragi ltaragi commented Sep 4, 2024

Description

Cherry-pick following commits from 2.x

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.

…ct#15624 to 2.x (opensearch-project#15595)

* [SnapshotV2] Snapshot Status API changes (opensearch-project#15409)

---------

Signed-off-by: Lakshya Taragi <[email protected]>
(cherry picked from commit 925f41b)

* Remove breaking changes from opensearch-project#15409 (opensearch-project#15624)

Signed-off-by: Lakshya Taragi <[email protected]>
(cherry picked from commit a60b668)
(cherry picked from commit 01b38be)
…ject#15642)

Signed-off-by: Lakshya Taragi <[email protected]>
(cherry picked from commit 0f53bf9)
(cherry picked from commit d368e3b)
@ltaragi ltaragi changed the title [SnapshotV2] Snapshot Status API changes [Backport] Snapshot Status API changes Sep 4, 2024
Copy link
Contributor

github-actions bot commented Sep 4, 2024

✅ Gradle check result for bcd700a: SUCCESS

Copy link

codecov bot commented Sep 4, 2024

Codecov Report

Attention: Patch coverage is 6.86275% with 95 lines in your changes missing coverage. Please review.

Project coverage is 71.63%. Comparing base (175cbd0) to head (bcd700a).
Report is 1 commits behind head on 2.17.

Files with missing lines Patch % Lines
...apshots/status/TransportSnapshotsStatusAction.java 0.00% 72 Missing ⚠️
...uster/snapshots/status/SnapshotsStatusRequest.java 20.00% 12 Missing ⚠️
...shots/TooManyShardsInSnapshotsStatusException.java 0.00% 7 Missing ⚠️
...napshots/status/SnapshotsStatusRequestBuilder.java 0.00% 4 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               2.17   #15665      +/-   ##
============================================
- Coverage     71.66%   71.63%   -0.03%     
+ Complexity    64220    64159      -61     
============================================
  Files          5250     5251       +1     
  Lines        300258   300341      +83     
  Branches      43682    43700      +18     
============================================
- Hits         215174   215152      -22     
- Misses        67084    67177      +93     
- Partials      18000    18012      +12     

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

@ltaragi
Copy link
Contributor Author

ltaragi commented Sep 4, 2024

Conflicts in following files while cherry-picking #15595 :

CHANGELOG.md
server/src/internalClusterTest/java/org/opensearch/snapshots/SnapshotStatusApisIT.java

@sachinpkale sachinpkale merged commit eaa7295 into opensearch-project:2.17 Sep 4, 2024
59 of 60 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants