Problem:

I was facing an issue where if elastic server connection was failing I would get an exception "No alive nodes found in your cluster" and the page would not load. So instead Im checking if cluster health is ok & returning an error if connection is down.

CommentFileSizeAuthor
#3 elastic_search-cluster-connection-3127438.patch3.61 KBsaurabhpandit26
#2 elastic_connector-check-connection-3127438.patch3.03 KBsaurabhpandit26

Issue fork elasticsearch_connector-3127438

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

saurabhpandit26 created an issue. See original summary.

saurabhpandit26’s picture

saurabhpandit26 commented
StatusFileSize
new elastic_connector-check-connection-3127438.patch3.03 KB
saurabhpandit26’s picture

saurabhpandit26 commented
StatusFileSize
new elastic_search-cluster-connection-3127438.patch3.61 KB

Im adding additional conditions when cluster config is not defined. These cause site-wide failure when used with modules like facets.

Letharion’s picture

Letharion commented

This patch saved me. I accidentally removed my cluster, and brought the whole site down as every page was just the exception.

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-5
commented
Status: Active » Needs review

There's a patch here, so I'm going to move this to "Needs review"

mparker17 opened merge request !146

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-5
commented

I just released elasticsearch_connector-8.x-7.0-alpha6, which had some changes that could break older patches, so I've been proactively fixing merge conflicts, hopefully saving people time when they need to upgrade.

I created merge request !146 fromt he patch in #3, and resolved the merge conflicts. If you need an updated patch file, you can download one from the merge request.

Thank you!