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

doc/log enhancement: disk watermark #10588

Closed
faxm0dem opened this issue Apr 14, 2015 · 2 comments · Fixed by #11313
Closed

doc/log enhancement: disk watermark #10588

faxm0dem opened this issue Apr 14, 2015 · 2 comments · Fixed by #11313
Labels
:Distributed/Distributed A catch all label for anything in the Distributed Area. If you aren't sure, use this one. good first issue low hanging fruit help wanted adoptme

Comments

@faxm0dem
Copy link

The documentation and key name of the cluster setting cluster.routing.allocation.disk.watermark.low is inconsistent with the message from the logfile. When value is set to default of 85% (according to documentation), the warning message says:

low disk watermark [15%] exceeded

It is IMHO misleading

@dakrone
Copy link
Member

dakrone commented Apr 14, 2015

You're right, this should be pretty easy to fix by either standardizing on displaying the used space or free space everywhere.

mkis- added a commit to mkis-/elasticsearch that referenced this issue May 19, 2015
documentaiton

Change log entries, add convenience methods, unit tests
@mkis-
Copy link
Contributor

mkis- commented May 19, 2015

Hello! I've attempted a fix for this by changing the logging so that it displays used space when dealing with percentages.

I've also created a pull request.

mkis- added a commit to mkis-/elasticsearch that referenced this issue Jun 1, 2015
documentaiton

Change log entries, add convenience methods, unit tests

Correct allocation decision messages to display used space

Missed a word

Correct displayed threshold value
dakrone pushed a commit that referenced this issue Jun 1, 2015
documentaiton

Change log entries, add convenience methods, unit tests

Correct allocation decision messages to display used space

Missed a word

Correct displayed threshold value
@lcawl lcawl added :Distributed/Distributed A catch all label for anything in the Distributed Area. If you aren't sure, use this one. and removed :Allocation labels Feb 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed/Distributed A catch all label for anything in the Distributed Area. If you aren't sure, use this one. good first issue low hanging fruit help wanted adoptme
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants