search

elastic / app-search-php

Elastic App Search Official PHP Client
https://www.elastic.co/products/app-search
Apache License 2.0
51 stars 21 forks source link

Fix CI tests for 7.8.0 #40

Closed richkuz closed 4 years ago

richkuz commented 4 years ago

Fixes a regression in CI integration tests caused by upgrading Enterprise Search and Elasticsearch to 7.8.0 in https://github.com/elastic/app-search-php/commit/bc33eebcb9f909d857a7ac1c085a70609544252f

The CI tests fail with:

--------------------------------------------------------------------------------
Elasticsearch API key service must be enabled. It is enabled automatically when you configure Elasticsearch to use TLS on the HTTP interface.
Alternatively, you can explicitly enable the setting within Elasticsearch by opening config/elasticsearch.yml and adding:

xpack.security.authc.api_key.enabled: true
--------------------------------------------------------------------------------

This PR adds that setting to Elasticsearch.

richkuz commented 4 years ago

The tests are passing now, but the CI test, DocumentApiTest::testDeleteDocuments(), is flaky. See https://github.com/elastic/app-search-php/issues/41