New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 1 user

Issue metadata

Status: Released
Owner: ----
Closed: Jul 4
Cc:
Components:



Sign in to add a comment

Improve the way of configuring elasticsearch in clustered mode

Project Member Reported by david.pu...@gmail.com, Jul 2

Issue description

*****************************************************************
*****                                                       *****
***** !!!! THIS BUG TRACKER IS FOR GERRIT CODE REVIEW !!!!  *****
*****                                                       *****
***** DO NOT SUBMIT BUGS FOR CHROME, ANDROID, CYANOGENMOD,  *****
***** INTERNAL ISSUES WITH YOUR COMPANY'S GERRIT SETUP, ETC.*****
*****                                                       *****
*****   THOSE ISSUES BELONG IN DIFFERENT ISSUE TRACKERS     *****
*****                                                       *****
*****************************************************************

Affected Version: 2.14.9

What steps will reproduce the problem?
1. Run Elasticsearch in clustered mode
2. Configure Gerrit to use it
3.

What is the expected output?

It should be simple to configure

What do you see instead?

For each ES server you have to specify a separate section with host, port and protocol in separate items

[elasticsearch]
   username = u
   password = p
 
 [elasticsearch "host1"]
   hostname = es1
   port = 1111
   protocol = https
 
 [elasticsearch "host2"]
   hostname = es2
   port = 1111
   protocol = https
 
 [elasticsearch "host3"]
   hostname = es3
   port = 1111
   protocol = https

Please provide any additional information below.

When the configuration is read, each section is parsed into a server represented by a HttpHost instantiated using the provided protocol, hostname and port. This could just as easily be done if the servers were provided in a list, i.e.:

[elasticsearch]
   username = u
   password = p
   server = https://es1:1111
   server = https://es2:1111
   server = https://es3:1111
 
Status: ChangeUnderReview (was: New)
https://gerrit-review.googlesource.com/c/gerrit/+/187213
Labels: FixedIn-2.14.10
Status: Submitted (was: ChangeUnderReview)
Status: Released (was: Submitted)

Sign in to add a comment