Skip to content

Conversation

@rubpa
Copy link

@rubpa rubpa commented Aug 29, 2019

Updated force-merge and freeze index API pages to clarify read-only indices.

rubpa added 3 commits August 29, 2019 09:46
Make it obvious that a frozen index is a read-only index.
Update forcemerge.asciidoc
@polyfractal polyfractal added :Data Management/Indices APIs APIs to create and manage indices and templates >docs General docs changes labels Aug 30, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs

@gwbrown
Copy link
Contributor

gwbrown commented Sep 4, 2019

Thanks for your PR @rubpa. Two things:

  1. Could you sign our contributor license agreement? This is necessary before we'll be able to merge your changes.
  2. I'm not sure the change to the force merge documentation is the right change to make because freezing an index is not the only way to make it read-only - for example, setting index.blocks.write will also make an index read-only.

@rubpa
Copy link
Author

rubpa commented Sep 6, 2019

Hi @gwbrown! While trying to manage space used by indices and finding all the new options in Kibana, I came across read-only used in the documentation but could not find any page to make an index read-only. After some digging, I found that freeze does it. Then the nice edit button gave me the thought of submitting this doc improvement as a PR.

So, it's not clear how to make read-only indices in the current documentation and I request you to do the most appropriate updates.

I saw the CLA while submitting the PR - still figuring out if I can sign it. I submitted this improvement in my personal capacity but I use elastic stack at work.

@jrodewig
Copy link
Contributor

Closing as the CLA is not signed, and the force merge changes are not clear.

However, I have opened #50092 to better clarify that frozen indices are read-only.

Thanks for raising this @rubpa!

@jrodewig jrodewig closed this Dec 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Data Management/Indices APIs APIs to create and manage indices and templates >docs General docs changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants