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/backport 15131 to 2.x #15673

Conversation

SwethaGuptha
Copy link
Contributor

Description

Backport #15131 to 2.17

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.

Gankris96 and others added 9 commits September 3, 2024 20:43
…#15579) (opensearch-project#15623)

* Abstract out RemoteFsTranslog with pinned timestamps


(cherry picked from commit deeb2de)

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>
…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)
…5545) (opensearch-project#15634)

This test assumed that the order of returned hits will match the order
of insertion. That's not generally true, especially if there was a
flush partway through, so documents end up in different segments.

This fixes it by explicitly sorting the returned documents to
guarantee that they come back in the correct order.

Also, we were getting a NPE when trying to output the failure message
because the expected value was intentionally null. I fixed that too.


(cherry picked from commit eb1cbb8)

Signed-off-by: Michael Froh <[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>
…imeout & Fix responsibility check for existing shards allocator when timed out (opensearch-project#15648)

* Optimise unassigned shards iteration after allocator timeout (opensearch-project#14977)

Signed-off-by: Rishab Nahata <[email protected]>
@SwethaGuptha SwethaGuptha changed the base branch from main to 2.17 September 4, 2024 11:48
Copy link
Contributor

github-actions bot commented Sep 4, 2024

❌ Gradle check result for d368e3b: null

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?

@opensearch-trigger-bot
Copy link
Contributor

This PR is stalled because it has been open for 30 days with no activity.

@opensearch-trigger-bot opensearch-trigger-bot bot added the stalled Issues that have stalled label Oct 4, 2024
@dblock
Copy link
Member

dblock commented Oct 9, 2024

This backport is into 2.17 that has shipped, closing.

@dblock dblock closed this Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stalled Issues that have stalled
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants