Skip to content

[Backport 2.16] Add perms for remote snapshot cache eviction on scripted query#14885

Merged
dbwiddis merged 1 commit into2.16from
backport/backport-14411-to-2.16
Jul 23, 2024
Merged

[Backport 2.16] Add perms for remote snapshot cache eviction on scripted query#14885
dbwiddis merged 1 commit into2.16from
backport/backport-14411-to-2.16

Conversation

@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor

@opensearch-trigger-bot opensearch-trigger-bot bot commented Jul 23, 2024

Backport 90d5500 from #14411.

@github-actions
Add perms for remote snapshot cache eviction on scripted query (#14411)
f626b87 
Signed-off-by: Finn Carroll <carrofin@amazon.com>
(cherry picked from commit 90d5500)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 23, 2024

❌ Gradle check result for f626b87: ABORTED

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 Jul 23, 2024

❌ Gradle check result for f626b87: 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 Jul 23, 2024

❌ Gradle check result for f626b87: 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 Jul 23, 2024

✅ Gradle check result for f626b87: SUCCESS

@codecov
Copy link
Copy Markdown

codecov bot commented Jul 23, 2024

Codecov Report

Attention: Patch coverage is 90.47619% with 2 lines in your changes missing coverage. Please review.

Please upload report for BASE (2.16@1d634b4). Learn more about missing BASE report.

Files Patch % Lines
...arch/index/store/remote/utils/TransferManager.java 90.47% 0 Missing and 2 partials ⚠️
Additional details and impacted files 
@@           Coverage Diff           @@
##             2.16   #14885   +/-   ##
=======================================
  Coverage        ?   71.55%           
  Complexity      ?    62523           
=======================================
  Files           ?     5110           
  Lines           ?   293757           
  Branches        ?    42805           
=======================================
  Hits            ?   210199           
  Misses          ?    65861           
  Partials        ?    17697

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

@dbwiddis dbwiddis merged commit b5cd3f5 into 2.16 Jul 23, 2024
@dbwiddis dbwiddis deleted the backport/backport-14411-to-2.16 branch July 23, 2024 15:42
@opensearch-ci-bot opensearch-ci-bot mentioned this pull request Sep 11, 2024
Open
@opensearch-ci-bot opensearch-ci-bot mentioned this pull request Sep 11, 2024
Open
@opensearch-ci-bot opensearch-ci-bot mentioned this pull request Sep 9, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dbwiddis dbwiddis dbwiddis approved these changes

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

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

@ashking94 ashking94 Awaiting requested review from ashking94 ashking94 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

@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

@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.

1 participant

@dbwiddis