diff --git a/build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/RestrictedBuildApiService.java b/build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/RestrictedBuildApiService.java index f0aff29d42aa..c7da5683ee41 100644 --- a/build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/RestrictedBuildApiService.java +++ b/build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/RestrictedBuildApiService.java @@ -36,12 +36,6 @@ public abstract class RestrictedBuildApiService implements BuildService parameters() throws Exception { return ESClientYamlSuiteTestCase.createParameters(); } + + @Override + protected String getTestRestCluster() { + return cluster.getHttpAddresses(); + } } diff --git a/test/external-modules/error-query/build.gradle b/test/external-modules/error-query/build.gradle index a66242290247..17cf6f4165fe 100644 --- a/test/external-modules/error-query/build.gradle +++ b/test/external-modules/error-query/build.gradle @@ -7,18 +7,11 @@ * License v3.0 only", or the "Server Side Public License, v 1". */ -apply plugin: 'elasticsearch.legacy-yaml-rest-test' +import org.elasticsearch.gradle.testclusters.StandaloneRestIntegTestTask + +apply plugin: 'elasticsearch.internal-yaml-rest-test' apply plugin: 'elasticsearch.internal-java-rest-test' -tasks.named('javaRestTest') { - usesDefaultDistribution("to be triaged") - it.onlyIf("snapshot build") { buildParams.snapshotBuild } -} - -tasks.named('yamlRestTest').configure { - it.onlyIf("snapshot build") { buildParams.snapshotBuild } -} - esplugin { description = 'A test module that exposes a way to simulate search shard failures and warnings' classname ='org.elasticsearch.test.errorquery.ErrorQueryPlugin' @@ -29,3 +22,14 @@ restResources { include '_common', 'indices', 'index', 'cluster', 'search' } } + +dependencies { + clusterModules project(':x-pack:plugin:esql') + clusterModules project(':x-pack:plugin:autoscaling') + clusterModules project(':x-pack:plugin:ilm') +} + +tasks.withType(StandaloneRestIntegTestTask) { + def isSnapshot = buildParams.snapshotBuild + onlyIf("snapshot build") { isSnapshot } +} diff --git a/test/external-modules/error-query/src/javaRestTest/java/org/elasticsearch/test/esql/EsqlPartialResultsIT.java b/test/external-modules/error-query/src/javaRestTest/java/org/elasticsearch/test/esql/EsqlPartialResultsIT.java index e135dd8d2517..1a4299b0ce93 100644 --- a/test/external-modules/error-query/src/javaRestTest/java/org/elasticsearch/test/esql/EsqlPartialResultsIT.java +++ b/test/external-modules/error-query/src/javaRestTest/java/org/elasticsearch/test/esql/EsqlPartialResultsIT.java @@ -16,7 +16,6 @@ import org.elasticsearch.client.ResponseException; import org.elasticsearch.common.settings.Settings; import org.elasticsearch.common.xcontent.support.XContentMapValues; import org.elasticsearch.test.cluster.ElasticsearchCluster; -import org.elasticsearch.test.cluster.local.distribution.DistributionType; import org.elasticsearch.test.rest.ESRestTestCase; import org.junit.ClassRule; @@ -35,8 +34,9 @@ import static org.hamcrest.Matchers.lessThanOrEqualTo; public class EsqlPartialResultsIT extends ESRestTestCase { @ClassRule public static ElasticsearchCluster cluster = ElasticsearchCluster.local() - .distribution(DistributionType.DEFAULT) .module("test-error-query") + .module("x-pack-esql") + .module("x-pack-ilm") .setting("xpack.security.enabled", "false") .setting("xpack.license.self_generated.type", "trial") .setting("esql.query.allow_partial_results", "true") diff --git a/test/external-modules/error-query/src/yamlRestTest/java/org/elasticsearch/search/query/ErrorQueryClientYamlTestSuiteIT.java b/test/external-modules/error-query/src/yamlRestTest/java/org/elasticsearch/search/query/ErrorQueryClientYamlTestSuiteIT.java index bdcd10600b34..58c4326bf903 100644 --- a/test/external-modules/error-query/src/yamlRestTest/java/org/elasticsearch/search/query/ErrorQueryClientYamlTestSuiteIT.java +++ b/test/external-modules/error-query/src/yamlRestTest/java/org/elasticsearch/search/query/ErrorQueryClientYamlTestSuiteIT.java @@ -12,10 +12,16 @@ package org.elasticsearch.search.query; import com.carrotsearch.randomizedtesting.annotations.Name; import com.carrotsearch.randomizedtesting.annotations.ParametersFactory; +import org.elasticsearch.test.cluster.ElasticsearchCluster; import org.elasticsearch.test.rest.yaml.ClientYamlTestCandidate; import org.elasticsearch.test.rest.yaml.ESClientYamlSuiteTestCase; +import org.junit.ClassRule; public class ErrorQueryClientYamlTestSuiteIT extends ESClientYamlSuiteTestCase { + + @ClassRule + public static ElasticsearchCluster cluster = ElasticsearchCluster.local().module("test-error-query").build(); + public ErrorQueryClientYamlTestSuiteIT(@Name("yaml") ClientYamlTestCandidate testCandidate) { super(testCandidate); } @@ -24,4 +30,9 @@ public class ErrorQueryClientYamlTestSuiteIT extends ESClientYamlSuiteTestCase { public static Iterable parameters() throws Exception { return ESClientYamlSuiteTestCase.createParameters(); } + + @Override + protected String getTestRestCluster() { + return cluster.getHttpAddresses(); + } } diff --git a/test/test-clusters/src/main/java/org/elasticsearch/test/cluster/local/DefaultLocalClusterHandle.java b/test/test-clusters/src/main/java/org/elasticsearch/test/cluster/local/DefaultLocalClusterHandle.java index f2e2245e7321..6620c0f7af40 100644 --- a/test/test-clusters/src/main/java/org/elasticsearch/test/cluster/local/DefaultLocalClusterHandle.java +++ b/test/test-clusters/src/main/java/org/elasticsearch/test/cluster/local/DefaultLocalClusterHandle.java @@ -41,7 +41,7 @@ public class DefaultLocalClusterHandle implements LocalClusterHandle { public static final AtomicInteger NEXT_DEBUG_PORT = new AtomicInteger(5007); private static final Logger LOGGER = LogManager.getLogger(DefaultLocalClusterHandle.class); - private static final Duration CLUSTER_UP_TIMEOUT = Duration.ofMinutes(5); + private static final Duration CLUSTER_UP_TIMEOUT = Duration.ofMinutes(1); public final ForkJoinPool executor = new ForkJoinPool( Math.max(Runtime.getRuntime().availableProcessors(), 4), diff --git a/test/yaml-rest-runner/build.gradle b/test/yaml-rest-runner/build.gradle index 37d2a00a68dd..1ae1315ac9ef 100644 --- a/test/yaml-rest-runner/build.gradle +++ b/test/yaml-rest-runner/build.gradle @@ -1,6 +1,6 @@ apply plugin: 'elasticsearch.build' -apply plugin: 'elasticsearch.legacy-yaml-rest-test' -apply plugin: 'elasticsearch.legacy-yaml-rest-compat-test' +apply plugin: 'elasticsearch.internal-yaml-rest-test' +apply plugin: 'elasticsearch.yaml-rest-compat-test' dependencies { api project(':test:framework') diff --git a/test/yaml-rest-runner/src/yamlRestTest/java/org/elasticsearch/test/rest/yaml/ESClientYamlSuiteTestCaseFailLogIT.java b/test/yaml-rest-runner/src/yamlRestTest/java/org/elasticsearch/test/rest/yaml/ESClientYamlSuiteTestCaseFailLogIT.java index abd70d5c3a32..708f3d8e5293 100644 --- a/test/yaml-rest-runner/src/yamlRestTest/java/org/elasticsearch/test/rest/yaml/ESClientYamlSuiteTestCaseFailLogIT.java +++ b/test/yaml-rest-runner/src/yamlRestTest/java/org/elasticsearch/test/rest/yaml/ESClientYamlSuiteTestCaseFailLogIT.java @@ -13,12 +13,17 @@ import com.carrotsearch.randomizedtesting.annotations.ParametersFactory; import org.apache.logging.log4j.Level; import org.elasticsearch.test.MockLog; +import org.elasticsearch.test.cluster.ElasticsearchCluster; import org.elasticsearch.test.junit.annotations.TestLogging; +import org.junit.ClassRule; import java.io.IOException; public class ESClientYamlSuiteTestCaseFailLogIT extends ESClientYamlSuiteTestCase { + @ClassRule + public static ElasticsearchCluster cluster = ElasticsearchCluster.local().build(); + public ESClientYamlSuiteTestCaseFailLogIT(final ClientYamlTestCandidate testCandidate) { super(testCandidate); } @@ -65,4 +70,9 @@ public class ESClientYamlSuiteTestCaseFailLogIT extends ESClientYamlSuiteTestCas mockLog.assertAllExpectationsMatched(); } } + + @Override + protected String getTestRestCluster() { + return cluster.getHttpAddresses(); + } }