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

Prepare the plugin to an elasticsearch 7.17.24 upgrade #16

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

garaud
Copy link

@garaud garaud commented Oct 1, 2024

Description

Upgrade the plugin with a more recent 7.17.x Elasticsearch version.

Changes

  • Change the elasticsearch version to 7.17.24

  • Update the build.gradle adding this kind of line

    tasks.named("yamlRestTest").configure {
      systemProperty 'tests.security.manager', 'false'
    }
    

    in order to avoid the test compilation failure java.lang.RuntimeException: unable to install test security manager

  • upgrade Gradle to 8.10.2 (the previous 7.5.1 version raised an exception on the check task with an error which looked like java.lang.NoSuchMethodError: 'org.gradle.api.services.internal.BuildServiceProvider org.gradle.api.services.internal.BuildServiceRegistryInternal.consume(java.lang.String, java.lang.Class)')

  • add some content to the README file

How to reproduce

Install Java 17. I suggest you to use sdkman tool which can help you to install any JVM for several languages (Kotlin, Java, Clojure, ...).

If you have an old build, I can also suggest you to remove the Docker image related to the elasticsearch-plugin-debug container and also remove the local .gradle folder.

If you use sdkman and you installed for instance 17.0.12-tem (OpenJDK by Eclipse Temurin)

  • skd use java 17.0.12-tem
  • ./gradlew build which compiles the code and run the tests
  • docker compose up
  • you should see a geopoint-clustering-aggregation-7.17.24.0.zip in the folder build/distributions
  • http :9200/_cat/plugins to check the plugin

@garaud garaud requested a review from 5k4nd October 1, 2024 20:02
@garaud garaud changed the title Chore/sc 52456/prepare elasticsearch 7.17.24 upgrade Prepare the plugin to an elasticsearch 7.17.24 upgrade Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant