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] Ensure index templates are not applied to system indices #16518

Merged
merged 1 commit into from
Oct 29, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 1e7c122 from #16418.

* fix: ensure system indices are processed without templates

Signed-off-by: Pavan Yekbote <[email protected]>

* refactor: overloaded method for creating without templates

Signed-off-by: Pavan Yekbote <[email protected]>

* test: adding test to check call for notemplates on system index

Signed-off-by: Pavan Yekbote <[email protected]>

* refactor: cchange modifier to package private and add entry in changelog

Signed-off-by: Pavan Yekbote <[email protected]>

* test: adding IT test

Signed-off-by: Pavan Yekbote <[email protected]>

* refactor: remove UT and add private modifiers

Signed-off-by: Pavan Yekbote <[email protected]>

* refactor: spotless changes

Signed-off-by: Pavan Yekbote <[email protected]>

---------

Signed-off-by: Pavan Yekbote <[email protected]>
(cherry picked from commit 1e7c122)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Copy link
Contributor

✅ Gradle check result for e1a86fa: SUCCESS

Copy link

codecov bot commented Oct 29, 2024

Codecov Report

Attention: Patch coverage is 25.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 71.85%. Comparing base (c26a49e) to head (e1a86fa).
Report is 5 commits behind head on 2.x.

Files with missing lines Patch % Lines
...h/cluster/metadata/MetadataCreateIndexService.java 25.00% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##                2.x   #16518      +/-   ##
============================================
+ Coverage     71.83%   71.85%   +0.02%     
- Complexity    65318    65379      +61     
============================================
  Files          5312     5312              
  Lines        304985   304994       +9     
  Branches      44436    44437       +1     
============================================
+ Hits         219079   219161      +82     
+ Misses        67609    67544      -65     
+ Partials      18297    18289       -8     

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

@reta reta merged commit bdcfcd0 into 2.x Oct 29, 2024
60 of 61 checks passed
@github-actions github-actions bot deleted the backport/backport-16418-to-2.x branch October 29, 2024 22:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant