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

[UA] Remove unnecessary index block settings #208222

Merged
merged 4 commits into from
Jan 28, 2025

Conversation

afharo
Copy link
Member

@afharo afharo commented Jan 24, 2025

Summary

Follow up to #207114

The _create_from API now removes the index.block settings by default. We don't need to unset them anymore.

Checklist

@afharo afharo requested a review from a team as a code owner January 24, 2025 15:50
@afharo afharo self-assigned this Jan 24, 2025
@afharo afharo added Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc release_note:skip Skip the PR/issue when compiling release notes backport:prev-minor Backport to (9.0) the previous minor version (i.e. one version back from main) labels Jan 24, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-core (Team:Core)

Copy link
Contributor

@TinaHeiligers TinaHeiligers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed the default behavior from reviewing the code in es.
LGTM

@afharo afharo enabled auto-merge (squash) January 28, 2025 07:38
@elasticmachine
Copy link
Contributor

elasticmachine commented Jan 28, 2025

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #21 / Case View Page similar cases tab should render the similar cases table
  • [job] [logs] FTR Configs #44 / Cloud Security Posture Agentless Security Posture Integration Options should show cspm with agentless option
  • [job] [logs] FTR Configs #44 / Cloud Security Posture Agentless Security Posture Integration Options should show cspm with agentless option
  • [job] [logs] Jest Tests #21 / HoverableAvatar renders the tooltip when hovering

Metrics [docs]

✅ unchanged

History

cc @afharo

@afharo afharo merged commit 37eca30 into elastic:main Jan 28, 2025
9 checks passed
@afharo afharo deleted the ua/remove_unnecessary_options branch January 28, 2025 20:59
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/13019437702

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jan 28, 2025
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Jan 28, 2025
# Backport

This will backport the following commits from `main` to `8.x`:
- [[UA] Remove unnecessary index block settings
(#208222)](#208222)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Alejandro Fernández
Haro","email":"alejandro.haro@elastic.co"},"sourceCommit":{"committedDate":"2025-01-28T20:59:45Z","message":"[UA]
Remove unnecessary index block settings
(#208222)","sha":"37eca30f85553e1a7012b38112195e886112af32","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Core","release_note:skip","v9.0.0","backport:prev-minor"],"title":"[UA]
Remove unnecessary index block
settings","number":208222,"url":"https://github.com/elastic/kibana/pull/208222","mergeCommit":{"message":"[UA]
Remove unnecessary index block settings
(#208222)","sha":"37eca30f85553e1a7012b38112195e886112af32"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/208222","number":208222,"mergeCommit":{"message":"[UA]
Remove unnecessary index block settings
(#208222)","sha":"37eca30f85553e1a7012b38112195e886112af32"}}]}]
BACKPORT-->

Co-authored-by: Alejandro Fernández Haro <alejandro.haro@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (9.0) the previous minor version (i.e. one version back from main) release_note:skip Skip the PR/issue when compiling release notes Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants