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] Fix assertion failure while deleting remote backed index #14654

Merged
merged 1 commit into from
Jul 23, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 74230b7 from #14601.

Copy link
Contributor

github-actions bot commented Jul 5, 2024

❕ Gradle check result for a1006cc: 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 Jul 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.48%. Comparing base (788a7fe) to head (f3b4fd6).
Report is 11 commits behind head on 2.x.

Additional details and impacted files
@@             Coverage Diff              @@
##                2.x   #14654      +/-   ##
============================================
- Coverage     71.52%   71.48%   -0.05%     
- Complexity    62546    62597      +51     
============================================
  Files          5110     5118       +8     
  Lines        293684   293961     +277     
  Branches      42800    42834      +34     
============================================
+ Hits         210063   210143      +80     
- Misses        65978    66183     +205     
+ Partials      17643    17635       -8     

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

Signed-off-by: Sachin Kale <[email protected]>
(cherry picked from commit 74230b7)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@dblock dblock force-pushed the backport/backport-14601-to-2.x branch from a1006cc to f3b4fd6 Compare July 23, 2024 16:44
@dblock
Copy link
Member

dblock commented Jul 23, 2024

I rebased this, cc: @sachinpkale you want to watch the backports actually get merged, this was missed.

Copy link
Contributor

✅ Gradle check result for f3b4fd6: SUCCESS

@dblock dblock merged commit 7ff82bb into 2.x Jul 23, 2024
32 checks passed
kkewwei pushed a commit to kkewwei/OpenSearch that referenced this pull request Jul 24, 2024
…project#14601) (opensearch-project#14654)

(cherry picked from commit 74230b7)

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

1 participant