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

Allow indexing when cluster state is red but write index state is healthy #2477

Merged
merged 3 commits into from Jul 14, 2016

Conversation

bernd
Copy link
Member

@bernd bernd commented Jul 13, 2016

The cluster state can turn red even though the current write index is green. This might happen when running rotation/retention during an ES maintenance window where not all nodes are present.

For index related tasks we are now checking the state of the current write index (deflector) instead of the state for all Graylog managed indices.

The logs and UI are still showing the red cluster state to make sure the admin will be notified.

Refs #2429
Fixes #2371

…yellow

The cluster state can turn red even though the current write index is
green. This might happen when running rotation/retention during an ES
maintenance window where not all nodes are present.

For index related tasks we are now checking the state of the current
write index (deflector) instead of the state for all Graylog managed
indices.

The logs and UI are still showing the red cluster state to make sure the
admin will be notified.

Refs #2429
Fixes #2371
@bernd bernd added this to the 2.1.0 milestone Jul 13, 2016
if (isConnected() && isHealthy()) {
LOG.debug("Cluster is healthy again, unblocking waiting threads.");
if (isConnected() && isDeflectorHealthy()) {
LOG.debug("Cluster and write index is healthy again, unblocking waiting threads.");
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this should be "or".

I would remove the part about the cluster completely: "Write-active index is healthy again, unblocking waiting threads."

@joschi joschi self-assigned this Jul 13, 2016
@joschi
Copy link
Contributor

joschi commented Jul 13, 2016

LGTM except for those 2 wordings. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants