Kibana: Force merge warning about read-only index on management UI

Created on 20 Mar 2020  路  7Comments  路  Source: elastic/kibana

The confirmation dialog for a force merge through the index management UI still has the warning message based on the ES 6.3 docs which is a bit misleading because you can't force merge an index that is in read-only mode.
I think PR 20264 and issue 19872 are related to the last change.

The ES docs seem to be updated according to this issue and this PR

image

Index Management Elasticsearch UI bug good first issue

All 7 comments

Pinging @elastic/kibana-app-arch (Team:AppArch)

Pinging @elastic/es-ui (Team:Elasticsearch UI)

Hi @stsourlidakis @cjcenizal, I would like to contribute on this if it's still open

Yes, please! Thank you @vishalsharma14! Do you need any guidance?

@vishalsharma14 To clarify, the message that needs to be changed is located here: https://github.com/elastic/kibana/blob/master/x-pack/plugins/index_management/public/application/sections/home/index_list/index_actions_context_menu/index_actions_context_menu.js#L364

The new message should read:

Don't force-merge indices to which you're still writing, or to which you'll write again in the future. Instead, rely on the automatic background merge process to perform merges as needed to keep the index running smoothly. If you write to a force-merged index then its performance may become much worse.

This PR should target the master branch. Once you create your PR I'll automatically get pinged to provide a code review and can help you more at that point.

@cjcenizal Thanks! I'll work on it and create a PR soon.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

cafuego picture cafuego  路  3Comments

snide picture snide  路  3Comments

Ginja picture Ginja  路  3Comments

bhavyarm picture bhavyarm  路  3Comments

timroes picture timroes  路  3Comments