opendev/gerrit
Code Issues Proposed changes

Merge branch 'stable-3.0' into stable-3.1

Browse Source 
* stable-3.0:
  ElasticContainer: Upgrade V7_5 to elasticsearch 7.5.1
  ElasticContainer: Upgrade V6_8 to elasticsearch 6.8.6
  Documentation: Update elasticsearch tests on macOS

Change-Id: I538c39f4a5ebf727f2f3a54458d49a95c110f7f3
This commit is contained in:
Marco Miller 2019-12-21 14:22:42 -05:00
commit 1d7ce2d98c
3 changed files with 12 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
Documentation/dev-bazel.txt
View File

@ -341,8 +341,14 @@ The following values are currently supported for the group name:
=== Elasticsearch
Successfully running the Elasticsearch tests requires Docker, and
may require setting the local
link:https://www.elastic.co/guide/en/elasticsearch/reference/current/vm-max-map-count.html[virtual memory].
may require setting the local virtual memory on
link:https://www.elastic.co/guide/en/elasticsearch/reference/current/vm-max-map-count.html[linux] and
link:https://www.elastic.co/guide/en/elasticsearch/reference/current/docker.html#_set_vm_max_map_count_to_at_least_262144[macOS].
On macOS, if using link:https://docs.docker.com/docker-for-mac/[Docker Desktop],
the effective memory value can be set in the Preferences, under the Advanced tab.
The default value usually does not suffice and is causing premature container exits.
That default is currently 2 GB and should be set to at least 5 (GB).
If Docker is not available, the Elasticsearch tests will be skipped.
Note that Bazel currently does not show

4
javatests/com/google/gerrit/elasticsearch/ElasticContainer.java
View File

@ -51,7 +51,7 @@ public class ElasticContainer extends ElasticsearchContainer {
case V6_7:
return "blacktop/elasticsearch:6.7.2";
case V6_8:
return "blacktop/elasticsearch:6.8.5";
return "blacktop/elasticsearch:6.8.6";
case V7_0:
return "blacktop/elasticsearch:7.0.1";
case V7_1:
@ -63,7 +63,7 @@ public class ElasticContainer extends ElasticsearchContainer {
case V7_4:
return "blacktop/elasticsearch:7.4.2";
case V7_5:
return "blacktop/elasticsearch:7.5.0";
return "blacktop/elasticsearch:7.5.1";
}
throw new IllegalStateException("No tests for version: " + version.name());
}

4
tools/nongoogle.bzl
View File

@ -102,8 +102,8 @@ def declare_nongoogle_deps():
# and httpasyncclient as necessary.
maven_jar(
name = "elasticsearch-rest-client",
artifact = "org.elasticsearch.client:elasticsearch-rest-client:7.5.0",
sha1 = "62535b6fc3a4e943e88e7640eac22e29f03a696d",
artifact = "org.elasticsearch.client:elasticsearch-rest-client:7.5.1",
sha1 = "094c155906dc94146fc5adc344ea2c676d487cf2",
)
maven_jar(