Skip to content

[Backport 2.x] Fix NPE on restore searchable snapshot#13920

Merged
andrross merged 2 commits into2.xfrom
backport/backport-13911-to-2.x
Jun 4, 2024
Merged

[Backport 2.x] Fix NPE on restore searchable snapshot#13920
andrross merged 2 commits into2.xfrom
backport/backport-13911-to-2.x

Conversation

@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor

@opensearch-trigger-bot opensearch-trigger-bot bot commented May 31, 2024

Backport 9a87624 from #13911.

@github-actions @andrross
Fix NPE on restore searchable snapshot (#13911)
15b10db 
Signed-off-by: panguixin <panguixin@bytedance.com>
Signed-off-by: Andrew Ross <andrross@amazon.com>
Co-authored-by: Andrew Ross <andrross@amazon.com>
(cherry picked from commit 9a87624)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@andrross
Add constant for compatiblity
c9fe180 
Signed-off-by: Andrew Ross <andrross@amazon.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 1, 2024

❌ Gradle check result for 15b10db: 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?

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 1, 2024

❌ Gradle check result for c9fe180: 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?

@andrross
Copy link
Copy Markdown
Member

andrross commented Jun 2, 2024

❌ Gradle check result for c9fe180: FAILURE

#13600

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 2, 2024

❌ Gradle check result for c9fe180: 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?

@ahmadbbi
Copy link
Copy Markdown

ahmadbbi commented Jun 3, 2024

Hi @andrross, since it's a blocker for us at the moment, wanted to ask when would this fix gets released? we are using version 2.12.0, would a patch version get released for that?

@andrross
Copy link
Copy Markdown
Member

andrross commented Jun 3, 2024

❌ Gradle check result for c9fe180: null

#13473

@andrross
Copy link
Copy Markdown
Member

andrross commented Jun 3, 2024

Hi @andrross, since it's a blocker for us at the moment, wanted to ask when would this fix gets released? we are using version 2.12.0, would a patch version get released for that?

@ahmadbabaeimoghadam This fix will go into 2.15, which will be released by June 25 at the latest.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 3, 2024

❌ Gradle check result for c9fe180: 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?

@andrross
Copy link
Copy Markdown
Member

andrross commented Jun 4, 2024

❌ Gradle check result for c9fe180: FAILURE

#13828

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 4, 2024

✅ Gradle check result for c9fe180: SUCCESS

@andrross andrross merged commit de13aca into 2.x Jun 4, 2024
@github-actions github-actions bot deleted the backport/backport-13911-to-2.x branch June 4, 2024 16:33
@prudhvigodithi prudhvigodithi mentioned this pull request Jun 7, 2024
Closed
This was referenced Jun 13, 2024
Open
Closed
Closed
Open
Open
Open
Closed
Open
@prudhvigodithi prudhvigodithi mentioned this pull request Jun 14, 2024
Closed
kkewwei pushed a commit to kkewwei/OpenSearch that referenced this pull request Jul 24, 2024
@opensearch-trigger-bot @github-actions
@andrross @kkewwei
[Backport 2.x] Fix NPE on restore searchable snapshot (opensearch-pro…
6d33e15 
…ject#13920)

* Fix NPE on restore searchable snapshot (opensearch-project#13911)

Signed-off-by: panguixin <panguixin@bytedance.com>
Signed-off-by: Andrew Ross <andrross@amazon.com>
Co-authored-by: Andrew Ross <andrross@amazon.com>
(cherry picked from commit 9a87624)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* Add constant for compatiblity

Signed-off-by: Andrew Ross <andrross@amazon.com>

---------

Signed-off-by: panguixin <panguixin@bytedance.com>
Signed-off-by: Andrew Ross <andrross@amazon.com>
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>
Co-authored-by: Andrew Ross <andrross@amazon.com>
Signed-off-by: kkewwei <kkewwei@163.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@andrross andrross andrross approved these changes

@anasalkouz anasalkouz Awaiting requested review from anasalkouz anasalkouz is a code owner

@Bukhtawar Bukhtawar Awaiting requested review from Bukhtawar Bukhtawar is a code owner

@CEHENKLE CEHENKLE Awaiting requested review from CEHENKLE CEHENKLE is a code owner

@dblock dblock Awaiting requested review from dblock

@dbwiddis dbwiddis Awaiting requested review from dbwiddis dbwiddis is a code owner

@dreamer-89 dreamer-89 Awaiting requested review from dreamer-89

@gbbafna gbbafna Awaiting requested review from gbbafna gbbafna is a code owner

@kotwanikunal kotwanikunal Awaiting requested review from kotwanikunal kotwanikunal is a code owner

@mch2 mch2 Awaiting requested review from mch2 mch2 is a code owner

@msfroh msfroh Awaiting requested review from msfroh msfroh is a code owner

@nknize nknize Awaiting requested review from nknize

@owaiskazi19 owaiskazi19 Awaiting requested review from owaiskazi19 owaiskazi19 is a code owner

@reta reta Awaiting requested review from reta reta is a code owner

@Rishikesh1159 Rishikesh1159 Awaiting requested review from Rishikesh1159 Rishikesh1159 is a code owner

@sachinpkale sachinpkale Awaiting requested review from sachinpkale sachinpkale is a code owner

@saratvemulapalli saratvemulapalli Awaiting requested review from saratvemulapalli saratvemulapalli is a code owner

@shwetathareja shwetathareja Awaiting requested review from shwetathareja shwetathareja is a code owner

@sohami sohami Awaiting requested review from sohami sohami is a code owner

@tlfeng tlfeng Awaiting requested review from tlfeng

@VachaShah VachaShah Awaiting requested review from VachaShah

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@andrross @ahmadbbi