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

HSEARCH-5288 Upgrade to Elasticsearch client 8.17.0 / HSEARCH-5289 Add Elasticsearch 8.17.0 compatibility / HSEARCH-5290 Test against latest Elasticsearch 7.17.26 #4429

Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions Jenkinsfile
Original file line number Diff line number Diff line change
@@ -251,7 +251,7 @@ stage('Configure') {
// so we don't test them
// See https://hibernate.atlassian.net/browse/HSEARCH-4340
new LocalElasticsearchBuildEnvironment(version: '7.16.3', condition: TestCondition.ON_DEMAND),
new LocalElasticsearchBuildEnvironment(version: '7.17.25', condition: TestCondition.AFTER_MERGE),
new LocalElasticsearchBuildEnvironment(version: '7.17.26', condition: TestCondition.AFTER_MERGE),
// Not testing 8.0 because we know there are problems in 8.0.1 (see https://hibernate.atlassian.net/browse/HSEARCH-4497)
// Not testing 8.1-8.6 to make the build quicker.
new LocalElasticsearchBuildEnvironment(version: '8.1.3', condition: TestCondition.ON_DEMAND),
@@ -269,7 +269,8 @@ stage('Configure') {
new LocalElasticsearchBuildEnvironment(version: '8.13.4', condition: TestCondition.ON_DEMAND),
new LocalElasticsearchBuildEnvironment(version: '8.14.3', condition: TestCondition.ON_DEMAND),
new LocalElasticsearchBuildEnvironment(version: '8.15.4', condition: TestCondition.ON_DEMAND),
new LocalElasticsearchBuildEnvironment(version: '8.16.1', condition: TestCondition.BEFORE_MERGE, isDefault: true),
new LocalElasticsearchBuildEnvironment(version: '8.16.1', condition: TestCondition.ON_DEMAND),
new LocalElasticsearchBuildEnvironment(version: '8.17.0', condition: TestCondition.BEFORE_MERGE, isDefault: true),
// IMPORTANT: Make sure to update the documentation for any newly supported Elasticsearch versions
// See version.org.elasticsearch.compatible.expected.text
// and version.org.elasticsearch.compatible.regularly-tested.text in POMs.
Original file line number Diff line number Diff line change
@@ -196,7 +196,7 @@ private ElasticsearchProtocolDialect createProtocolDialectElasticV7(Elasticsearc
}

private ElasticsearchProtocolDialect createProtocolDialectElasticV8(ElasticsearchVersion version, int minor) {
if ( minor > 16 ) {
if ( minor > 17 ) {
VersionLog.INSTANCE.unknownElasticsearchVersion( version );
}
else if ( minor == 0 ) {
Original file line number Diff line number Diff line change
@@ -281,14 +281,22 @@ public static List<? extends Arguments> params() {
ElasticsearchDistributionName.ELASTIC, "8.16.0", "8.16.0",
Elasticsearch814ModelDialect.class, Elasticsearch81ProtocolDialect.class
),
successWithWarning(
success(
ElasticsearchDistributionName.ELASTIC, "8.17", "8.17.0",
Elasticsearch814ModelDialect.class, Elasticsearch81ProtocolDialect.class
),
successWithWarning(
success(
ElasticsearchDistributionName.ELASTIC, "8.17.0", "8.17.0",
Elasticsearch814ModelDialect.class, Elasticsearch81ProtocolDialect.class
),
successWithWarning(
ElasticsearchDistributionName.ELASTIC, "8.18", "8.18.0",
Elasticsearch814ModelDialect.class, Elasticsearch81ProtocolDialect.class
),
successWithWarning(
ElasticsearchDistributionName.ELASTIC, "8.18.0", "8.18.0",
Elasticsearch814ModelDialect.class, Elasticsearch81ProtocolDialect.class
),
successWithWarning(
ElasticsearchDistributionName.ELASTIC, "9.0.0", "9.0.0",
Elasticsearch814ModelDialect.class, Elasticsearch81ProtocolDialect.class
2 changes: 1 addition & 1 deletion build/container/search-backend/elastic.Dockerfile
Original file line number Diff line number Diff line change
@@ -5,4 +5,4 @@
# * update `version.org.elasticsearch.latest` property in a POM file.
# * update the tags for 'elasticsearch-current' and 'elasticsearch-next' builds in ci/dependency-update/Jenkinsfile
#
FROM docker.io/elastic/elasticsearch:8.16.1
FROM docker.io/elastic/elasticsearch:8.17.0
4 changes: 2 additions & 2 deletions build/parents/build/pom.xml
Original file line number Diff line number Diff line change
@@ -48,15 +48,15 @@
<!-- >>> Elasticsearch -->
<!-- The version of the Elasticsearch client used by Hibernate Search, independently of the version of the remote cluster -->
<!-- Use the latest open-source version here. Currently, low-level clients are open-source even in 8.5+ -->
<version.org.elasticsearch.client>8.16.1</version.org.elasticsearch.client>
<version.org.elasticsearch.client>8.17.0</version.org.elasticsearch.client>
<!-- The main compatible version of Elasticsearch, advertised by default. Used in documentation links. -->
<version.org.elasticsearch.compatible.main>${version.org.elasticsearch.latest}</version.org.elasticsearch.compatible.main>
<documentation.org.elasticsearch.url>https://www.elastic.co/guide/en/elasticsearch/reference/${parsed-version.org.elasticsearch.compatible.main.majorVersion}.${parsed-version.org.elasticsearch.compatible.main.minorVersion}</documentation.org.elasticsearch.url>

<!-- The versions of Elasticsearch advertised as compatible with Hibernate Search -->
<!-- Make sure to only mention tested versions here -->
<!-- Make sure that 7.10 stays explicitly mentioned here, because that's the last open-source version -->
<version.org.elasticsearch.compatible.regularly-tested.text>7.10, 7.17 or 8.16</version.org.elasticsearch.compatible.regularly-tested.text>
<version.org.elasticsearch.compatible.regularly-tested.text>7.10, 7.17 or 8.17</version.org.elasticsearch.compatible.regularly-tested.text>
<!-- These are the versions same as above, but pointing only to the major part (used in compatibility section of ES backend documentation
as versions that Hibernate Search is compatible with. -->
<!-- NOTE: Adding new major versions would require to update the compatibility table in `backend-elasticsearch-compatibility` section of `backend-elasticsearch.asciidoc`. -->
5 changes: 2 additions & 3 deletions ci/dependency-update/Jenkinsfile
Original file line number Diff line number Diff line change
@@ -46,8 +46,7 @@ Map settings() {
updateProperties: [],
onlyRunTestDependingOn: ['hibernate-search-backend-elasticsearch'],
// We want to use the snapshot version of an image from the ES registry since that's where they are publishing their snapshots.
additionalMavenArgs: '-Dtest.lucene.skip=true -Dtest.elasticsearch.run.elastic.image.name=docker.elastic.co/elasticsearch/elasticsearch -Dtest.elasticsearch.run.elastic.image.tag=8.16.2-SNAPSHOT',
// This job won't change the versions in the pom. We are passing the latest Elasticsearch version through an additional maven argument `-D`
additionalMavenArgs: '-Dtest.lucene.skip=true -Dtest.elasticsearch.run.elastic.image.name=docker.elastic.co/elasticsearch/elasticsearch -Dtest.elasticsearch.run.elastic.image.tag=8.17.1-SNAPSHOT', // This job won't change the versions in the pom. We are passing the latest Elasticsearch version through an additional maven argument `-D`
skipSourceModifiedCheck: true
]
// Targets the next major/minor release of the of Elasticsearch
@@ -58,7 +57,7 @@ Map settings() {
updateProperties: [],
onlyRunTestDependingOn: ['hibernate-search-backend-elasticsearch'],
// We want to use the snapshot version of an image from the ES registry since that's where they are publishing their snapshots.
additionalMavenArgs: '-Dtest.lucene.skip=true -Dtest.elasticsearch.run.elastic.image.name=docker.elastic.co/elasticsearch/elasticsearch -Dtest.elasticsearch.run.elastic.image.tag=8.17.0-SNAPSHOT',
additionalMavenArgs: '-Dtest.lucene.skip=true -Dtest.elasticsearch.run.elastic.image.name=docker.elastic.co/elasticsearch/elasticsearch -Dtest.elasticsearch.run.elastic.image.tag=8.18.0-SNAPSHOT',
// This job won't change the versions in the pom. We are passing the latest Elasticsearch version through an additional maven argument `-D`
skipSourceModifiedCheck: true
]
2 changes: 1 addition & 1 deletion pom.xml
Original file line number Diff line number Diff line change
@@ -377,7 +377,7 @@

<!-- Container images for various integration tests -->
<!-- The latest version of Elasticsearch tested against by default -->
<version.org.elasticsearch.latest>8.16.1</version.org.elasticsearch.latest>
<version.org.elasticsearch.latest>8.17.0</version.org.elasticsearch.latest>
<test.elasticsearch.version></test.elasticsearch.version>
<test.elasticsearch.distribution>elastic</test.elasticsearch.distribution>

Loading