Add support for Elasticsearch version 7.8.*

Upgrade elasticsearch-rest-client to 7.8.0.

Add V7_8 to the version manager and use it for all the V7 tests.

Change-Id: I1474289bd5636ab30dcff6b14e78feddc3e8be60
This commit is contained in:
Marco Miller
2020-06-18 16:28:32 -04:00
parent ad273b556f
commit 7620adfa54
10 changed files with 18 additions and 9 deletions

View File

@@ -28,7 +28,8 @@ public enum ElasticVersion {
V7_4("7.4.*"), V7_4("7.4.*"),
V7_5("7.5.*"), V7_5("7.5.*"),
V7_6("7.6.*"), V7_6("7.6.*"),
V7_7("7.7.*"); V7_7("7.7.*"),
V7_8("7.8.*");
private final String version; private final String version;
private final Pattern pattern; private final Pattern pattern;

View File

@@ -31,7 +31,7 @@ public class ElasticReindexIT extends AbstractReindexTests {
@ConfigSuite.Config @ConfigSuite.Config
public static Config elasticsearchV7() { public static Config elasticsearchV7() {
return getConfig(ElasticVersion.V7_7); return getConfig(ElasticVersion.V7_8);
} }
@Override @Override

View File

@@ -31,7 +31,7 @@ public class ElasticIndexIT extends AbstractIndexTests {
@ConfigSuite.Config @ConfigSuite.Config
public static Config elasticsearchV7() { public static Config elasticsearchV7() {
return getConfig(ElasticVersion.V7_7); return getConfig(ElasticVersion.V7_8);
} }
@Override @Override

View File

@@ -60,6 +60,8 @@ public class ElasticContainer extends ElasticsearchContainer {
return "blacktop/elasticsearch:7.6.2"; return "blacktop/elasticsearch:7.6.2";
case V7_7: case V7_7:
return "blacktop/elasticsearch:7.7.1"; return "blacktop/elasticsearch:7.7.1";
case V7_8:
return "blacktop/elasticsearch:7.8.0";
} }
throw new IllegalStateException("No tests for version: " + version.name()); throw new IllegalStateException("No tests for version: " + version.name());
} }

View File

@@ -36,7 +36,7 @@ public class ElasticV7QueryAccountsTest extends AbstractQueryAccountsTest {
public static void startIndexService() { public static void startIndexService() {
if (container == null) { if (container == null) {
// Only start Elasticsearch once // Only start Elasticsearch once
container = ElasticContainer.createAndStart(ElasticVersion.V7_7); container = ElasticContainer.createAndStart(ElasticVersion.V7_8);
} }
} }

View File

@@ -42,7 +42,7 @@ public class ElasticV7QueryChangesTest extends AbstractQueryChangesTest {
public static void startIndexService() { public static void startIndexService() {
if (container == null) { if (container == null) {
// Only start Elasticsearch once // Only start Elasticsearch once
container = ElasticContainer.createAndStart(ElasticVersion.V7_7); container = ElasticContainer.createAndStart(ElasticVersion.V7_8);
client = HttpAsyncClients.createDefault(); client = HttpAsyncClients.createDefault();
client.start(); client.start();
} }

View File

@@ -36,7 +36,7 @@ public class ElasticV7QueryGroupsTest extends AbstractQueryGroupsTest {
public static void startIndexService() { public static void startIndexService() {
if (container == null) { if (container == null) {
// Only start Elasticsearch once // Only start Elasticsearch once
container = ElasticContainer.createAndStart(ElasticVersion.V7_7); container = ElasticContainer.createAndStart(ElasticVersion.V7_8);
} }
} }

View File

@@ -36,7 +36,7 @@ public class ElasticV7QueryProjectsTest extends AbstractQueryProjectsTest {
public static void startIndexService() { public static void startIndexService() {
if (container == null) { if (container == null) {
// Only start Elasticsearch once // Only start Elasticsearch once
container = ElasticContainer.createAndStart(ElasticVersion.V7_7); container = ElasticContainer.createAndStart(ElasticVersion.V7_8);
} }
} }

View File

@@ -54,6 +54,9 @@ public class ElasticVersionTest extends GerritBaseTests {
assertThat(ElasticVersion.forVersion("7.7.0")).isEqualTo(ElasticVersion.V7_7); assertThat(ElasticVersion.forVersion("7.7.0")).isEqualTo(ElasticVersion.V7_7);
assertThat(ElasticVersion.forVersion("7.7.1")).isEqualTo(ElasticVersion.V7_7); assertThat(ElasticVersion.forVersion("7.7.1")).isEqualTo(ElasticVersion.V7_7);
assertThat(ElasticVersion.forVersion("7.8.0")).isEqualTo(ElasticVersion.V7_8);
assertThat(ElasticVersion.forVersion("7.8.1")).isEqualTo(ElasticVersion.V7_8);
} }
@Test @Test
@@ -77,6 +80,7 @@ public class ElasticVersionTest extends GerritBaseTests {
assertThat(ElasticVersion.V7_5.isAtLeastMinorVersion(ElasticVersion.V6_7)).isFalse(); assertThat(ElasticVersion.V7_5.isAtLeastMinorVersion(ElasticVersion.V6_7)).isFalse();
assertThat(ElasticVersion.V7_6.isAtLeastMinorVersion(ElasticVersion.V6_7)).isFalse(); assertThat(ElasticVersion.V7_6.isAtLeastMinorVersion(ElasticVersion.V6_7)).isFalse();
assertThat(ElasticVersion.V7_7.isAtLeastMinorVersion(ElasticVersion.V6_7)).isFalse(); assertThat(ElasticVersion.V7_7.isAtLeastMinorVersion(ElasticVersion.V6_7)).isFalse();
assertThat(ElasticVersion.V7_8.isAtLeastMinorVersion(ElasticVersion.V6_7)).isFalse();
} }
@Test @Test
@@ -92,6 +96,7 @@ public class ElasticVersionTest extends GerritBaseTests {
assertThat(ElasticVersion.V7_5.isV6OrLater()).isTrue(); assertThat(ElasticVersion.V7_5.isV6OrLater()).isTrue();
assertThat(ElasticVersion.V7_6.isV6OrLater()).isTrue(); assertThat(ElasticVersion.V7_6.isV6OrLater()).isTrue();
assertThat(ElasticVersion.V7_7.isV6OrLater()).isTrue(); assertThat(ElasticVersion.V7_7.isV6OrLater()).isTrue();
assertThat(ElasticVersion.V7_8.isV6OrLater()).isTrue();
} }
@Test @Test
@@ -107,5 +112,6 @@ public class ElasticVersionTest extends GerritBaseTests {
assertThat(ElasticVersion.V7_5.isV7OrLater()).isTrue(); assertThat(ElasticVersion.V7_5.isV7OrLater()).isTrue();
assertThat(ElasticVersion.V7_6.isV7OrLater()).isTrue(); assertThat(ElasticVersion.V7_6.isV7OrLater()).isTrue();
assertThat(ElasticVersion.V7_7.isV7OrLater()).isTrue(); assertThat(ElasticVersion.V7_7.isV7OrLater()).isTrue();
assertThat(ElasticVersion.V7_8.isV7OrLater()).isTrue();
} }
} }

View File

@@ -94,8 +94,8 @@ def declare_nongoogle_deps():
# and httpasyncclient as necessary. # and httpasyncclient as necessary.
maven_jar( maven_jar(
name = "elasticsearch-rest-client", name = "elasticsearch-rest-client",
artifact = "org.elasticsearch.client:elasticsearch-rest-client:7.7.1", artifact = "org.elasticsearch.client:elasticsearch-rest-client:7.8.0",
sha1 = "6d44a8e35c11df6883747200bcf46f476a1782b8", sha1 = "ab28f6110bdc7d2ec886e1d6ff29a6c8ee30b883",
) )
maven_jar( maven_jar(