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.

Sign up for GitHub

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

Jump to bottom

Allow multiple ES hosts - in a cluster- to be specified #18870

Closed
wezell opened this issue Jul 13, 2020 · 3 comments
Closed

Allow multiple ES hosts - in a cluster- to be specified #18870

wezell opened this issue Jul 13, 2020 · 3 comments
Labels
Merged QA : Approved QA : Passed Internal Release : 5.3.5 Type : Enhancement

Comments

@wezell
Copy link
Contributor

wezell commented Jul 13, 2020 •
edited
Loading

When dotCMS connects to to elasticsearch via the high level Java client, we only allow for a single host to be specified. It turns out that Elasticsearch RestClientBuilder can accept multiple hosts that it will automatically load balance against and failover if needed. We need to allow this to allow for running against an elasticsearch cluster that does not require its own LB.

We will need to update our docker image to support this
wezell added a commit that referenced this issue Jul 13, 2020
@wezell
#18870 allowing multiple hosts, also ES_ENDPOINTS for a comma separat…
3827868 
…ed list of endpoints to try

'
wezell added a commit that referenced this issue Jul 13, 2020
@wezell
#18870 no reason to have multiple properties
9b7d72e
wezell added a commit that referenced this issue Jul 14, 2020
@wezell
#18870 logging
4400c0e
@wezell
Copy link
Contributor Author

wezell commented Jul 15, 2020

PR: #18877

@wezell wezell added this to the Bug Sprint milestone Jul 15, 2020
dsilvam pushed a commit that referenced this issue Jul 15, 2020
@wezell
Issue 18870 allow es hostnames array (#18877)
c46fdc8 
* #18870 allowing multiple hosts, also ES_ENDPOINTS for a comma separated list of endpoints to try
'

* #18870 no reason to have multiple properties

* #18870 logging
@dsilvam dsilvam self-assigned this Jul 22, 2020
@wezell wezell changed the title Allow multiple ES hosts to be specified Allow multiple ES hosts - in a cluster- to be specified Jul 22, 2020
@dsilvam
Copy link
Contributor

dsilvam commented Jul 22, 2020

Passed Internal QA: Started using the new property :
ES_ENDPOINTS=https://localhost:9200, https://localhost:9201, https://localhost:9202

Result:

Screen Shot 2020-07-22 at 5 06 28 PM

@dsilvam dsilvam removed their assignment Jul 22, 2020
@bryanboza
Copy link
Member

Fixed, tested on release-5.3.5 // Postgres // FF

@wezell wezell closed this as completed Jul 30, 2020
@brentgriffin brentgriffin mentioned this issue Aug 4, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Merged QA : Approved QA : Passed Internal Release : 5.3.5 Type : Enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@wezell @dsilvam @bryanboza