Skip to content

[Backport 2.x] [Snapshot V2] Move timestamp pinning before cluster state update#16325

Merged
dbwiddis merged 1 commit into2.xfrom
backport/backport-16269-to-2.x
Oct 15, 2024
Merged

[Backport 2.x] [Snapshot V2] Move timestamp pinning before cluster state update#16325
dbwiddis merged 1 commit into2.xfrom
backport/backport-16269-to-2.x

Conversation

@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor

Backport b5dcde3 from #16269.

)

* Move timestamp pinning before cluster state update

Signed-off-by: Gaurav Bafna <gbbafna@amazon.com>

* Address PR Comments

Signed-off-by: Gaurav Bafna <gbbafna@amazon.com>

* Fix IT

Signed-off-by: Gaurav Bafna <gbbafna@amazon.com>

---------

Signed-off-by: Gaurav Bafna <gbbafna@amazon.com>
(cherry picked from commit b5dcde3)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

✅ Gradle check result for e32454c: SUCCESS

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 15, 2024

Codecov Report

Attention: Patch coverage is 55.26316% with 17 lines in your changes missing coverage. Please review.

Project coverage is 71.79%. Comparing base (5947002) to head (e32454c).
Report is 12 commits behind head on 2.x.

Files with missing lines Patch % Lines
...ava/org/opensearch/snapshots/SnapshotsService.java 55.26% 15 Missing and 2 partials ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##                2.x   #16325      +/-   ##
============================================
+ Coverage     71.74%   71.79%   +0.04%     
- Complexity    65022    65054      +32     
============================================
  Files          5302     5303       +1     
  Lines        303866   303855      -11     
  Branches      44192    44189       -3     
============================================
+ Hits         218012   218146     +134     
+ Misses        67648    67520     -128     
+ Partials      18206    18189      -17     

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

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