- 🇨🇦Canada mparker17 UTC-4
I think when this issue was moved into the elasticsearch_connector queue, its version number wasn't updated, because the patch applies to 8.x-7.x. Updating the version.
- 🇩🇪Germany IT-Cru Munich
@sokru: Maybe it make sense to fix it first in 8.0.x-dev and than backport solution to 8.x-7.x-dev?
- Status changed to Needs work
3 months ago 9:15am 31 August 2024 - 🇫🇮Finland sokru
Actually still applies to 8.0.x-dev (and search_api_opensearch).
- 🇫🇮Finland sokru
The MR handles the error, but wonder if this needs a Functional test, something like https://git.drupalcode.org/project/search_api/-/blob/8.x-1.x/tests/src/F...
- Status changed to Needs review
3 months ago 9:20am 31 August 2024 - 🇫🇮Finland sokru
Added the tests, reverting the changes to
src/SearchAPI/BackendClient.php
will make the test fail.I added one @todo comment with link to upstream issue (https://github.com/elastic/elasticsearch-php/issues/1308) getting that done would help providing more informative error messages. Currently the wrong port will produce an error "Elastic\Transport\Exception\NoNodeAvailableException: No alive nodes. All the 1 nodes seem to be down", which is misleading.
The title of this issue could be better.
- Status changed to RTBC
about 2 months ago 5:51am 19 September 2024 - 🇨🇦Canada man-1982
Report:
1. I added bad Elasticsearch conf, which doesn work on my local env
2. I had this error
3. Applied patch through composer in composer patches part"drupal/elasticsearch_connector": { "produce fatal error when backend is down": "https://git.drupalcode.org/project/elasticsearch_connector/-/merge_requests/63.patch" }
4. Patch was applied without problem, all passed smoothly
5. as a result a can see list of my servers and index
I strongly recommend apply this changes to module.
It seems to me, there is a critical issue, because we don't have access to servers search list without this patchthanks to
@kallevu
@sokru - 🇫🇮Finland iSampo
MR63 working as it should, the error is no longer appearing with faulty URLs. Attached is a patch from the current MR63 changes to get a static patch for composer installs.
- Status changed to Fixed
about 2 months ago 5:59am 20 September 2024 - 🇫🇮Finland sokru
Committed to 8.0.x, not sure if I find a time to fix this also on 8.x-7.x.
Automatically closed - issue fixed for 2 weeks with no activity.