Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

[BUG]: Elasticsearch 7 based configuration fails to start the app #2447

Closed
buzz08 opened this issue Sep 5, 2021 · 2 comments
Closed

[BUG]: Elasticsearch 7 based configuration fails to start the app #2447

buzz08 opened this issue Sep 5, 2021 · 2 comments
Labels
type: bug bugs/ bug fixes

Comments

@buzz08
Copy link

buzz08 commented Sep 5, 2021

Describe the bug
When running the app with the ElasticSearch 7 configuration - it fails to start - Looks like there is a classpath conflict between the older version of ES and the newer version. This is the

Details
Conductor version: 3.3.0-SNAPSHOT (Latest from GitHub)

To Reproduce
Use the following properties:

conductor.elasticsearch.indexName=conductor
conductor.indexing.enabled=true
conductor.elasticsearch.version=7```

Run the app: 
java -jar -DCONDUCTOR_CONFIG_FILE=./conductor-config-overrides.properties ./server/build/libs/conductor-server-3.3.0-SNAPSHOT-boot.jar

**Expected behavior**
Server should have started without any errors and should have connected to the locally installed ElasticSearch version 7

**Output**
***************************
APPLICATION FAILED TO START
***************************

Description:

An attempt was made to call a method that does not exist. The attempt was made from the following location:

    com.netflix.conductor.es7.dao.index.ElasticSearchRestDAOV7.waitForHealthyCluster(ElasticSearchRestDAOV7.java:320)

The following method did not exist:

    'void org.elasticsearch.client.Request.addParameters(java.util.Map)'

The method's class, org.elasticsearch.client.Request, is available from the following locations:

    jar:file:/Users/boney/projects/examples/conductor/server/build/libs/conductor-server-3.3.0-SNAPSHOT-boot.jar!/BOOT-INF/lib/elasticsearch-rest-client-6.8.12.jar!/org/elasticsearch/client/Request.class

The class hierarchy was loaded from the following locations:

    org.elasticsearch.client.Request: jar:file:/Users/boney/projects/examples/conductor/server/build/libs/conductor-server-3.3.0-SNAPSHOT-boot.jar!/BOOT-INF/lib/elasticsearch-rest-client-6.8.12.jar!/


Action:

Correct the classpath of your application so that it contains a single, compatible version of org.elasticsearch.client.Request


**Additional context**
NA
@buzz08 buzz08 added the type: bug bugs/ bug fixes label Sep 5, 2021
@aravindanr
Copy link
Collaborator

aravindanr commented Sep 13, 2021

I can reproduce it. Looks like its related to #2324 , @FpyrypT can you look into this?

@buzz08 Unless you have a strict requirement for using ES7, the default supported version of ElasticSearch supported by Conductor is 6.x.

@buzz08
Copy link
Author

buzz08 commented Sep 13, 2021

Yes - managed to work with ES6 and also ES7 after changing some of the code per the instructions in the readme for ES7. So we can perhaps close this and wait for official support for ES7. Thanks for checking.

@buzz08 buzz08 closed this as completed Sep 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: bug bugs/ bug fixes
Projects
None yet
Development

No branches or pull requests

2 participants