Skip to content

[Backport 2.x] Add path prefix support to hashed prefix snapshots (#15664)#15758

Merged
ashking94 merged 3 commits intoopensearch-project:2.xfrom
ashking94:2.x-backport-15664
Sep 9, 2024
Merged

[Backport 2.x] Add path prefix support to hashed prefix snapshots (#15664)#15758
ashking94 merged 3 commits intoopensearch-project:2.xfrom
ashking94:2.x-backport-15664

Conversation

@ashking94
Copy link
Copy Markdown
Member

@ashking94 ashking94 commented Sep 5, 2024

Backport 245fa47 from #15664

@ashking94
Add path prefix support to hashed prefix snapshots (opensearch-projec…
ac456fc 
…t#15664)

* Add path prefix support to hashed prefix snapshots

Signed-off-by: Ashish Singh <ssashish@amazon.com>

* Fix test

Signed-off-by: Ashish Singh <ssashish@amazon.com>

---------

Signed-off-by: Ashish Singh <ssashish@amazon.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Sep 5, 2024

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

@ashking94
Merge branch '2.x' into 2.x-backport-15664
49cb230 
Signed-off-by: Ashish Singh <ssashish@amazon.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Sep 6, 2024

❌ Gradle check result for 49cb230: 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 Sep 6, 2024

❌ Gradle check result for 49cb230: 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 Sep 6, 2024

❌ Gradle check result for 49cb230: 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 Sep 6, 2024

❌ Gradle check result for 49cb230: 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 Sep 6, 2024

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

This was referenced Sep 6, 2024
Open
Closed
Open
Closed
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Sep 8, 2024

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

@ashking94
Merge remote-tracking branch 'upstream/2.x' into 2.x-backport-15664
a559172 
Signed-off-by: Ashish Singh <ssashish@amazon.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Sep 9, 2024

✅ Gradle check result for a559172: SUCCESS

@codecov
Copy link
Copy Markdown

codecov bot commented Sep 9, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.59%. Comparing base (19c204f) to head (a559172).
Report is 1 commits behind head on 2.x.

Additional details and impacted files 
@@             Coverage Diff              @@
##                2.x   #15758      +/-   ##
============================================
- Coverage     71.68%   71.59%   -0.10%     
+ Complexity    64487    64441      -46     
============================================
  Files          5263     5263              
  Lines        301684   301687       +3     
  Branches      43927    43927              
============================================
- Hits         216271   215993     -278     
- Misses        67311    67658     +347     
+ Partials      18102    18036      -66

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

@ashking94 ashking94 merged commit 1cdefc7 into opensearch-project:2.x Sep 9, 2024
This was referenced Sep 11, 2024
Open
Closed
Open
@prudhvigodithi prudhvigodithi mentioned this pull request Oct 16, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sachinpkale sachinpkale sachinpkale approved these changes

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

@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

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

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

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

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

3 participants

@ashking94 @sachinpkale @gbbafna