Skip to content

Commit 7304197

Browse files
authored
Move away from DistributionType.DEFAULT in downsample YAML tests (#133487)
1 parent b1b2121 commit 7304197

File tree

5 files changed

+32
-17
lines changed

5 files changed

+32
-17
lines changed

x-pack/plugin/downsample/build.gradle

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,10 +12,10 @@ base {
1212
}
1313

1414
dependencies {
15-
compileOnly project(path: xpackModule('core'))
15+
compileOnly project(xpackModule('core'))
1616
testImplementation project(':modules:data-streams')
17-
testImplementation project(path: xpackModule('ilm'))
18-
compileOnly project(path: xpackModule('mapper-aggregate-metric'))
17+
testImplementation project(xpackModule('ilm'))
18+
compileOnly project(xpackModule('mapper-aggregate-metric'))
1919
testImplementation(testArtifact(project(xpackModule('core'))))
2020
testImplementation project(xpackModule('ccr'))
2121
testImplementation project(xpackModule('esql'))

x-pack/plugin/downsample/qa/rest/build.gradle

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,17 @@ apply plugin: 'elasticsearch.yaml-rest-compat-test'
1010
apply plugin: 'elasticsearch.internal-test-artifact'
1111

1212
dependencies {
13-
testImplementation project(path: ':test:test-clusters')
14-
yamlRestTestImplementation project(path: xpackModule('rollup'))
13+
testImplementation project(':test:test-clusters')
14+
yamlRestTestImplementation project(xpackModule('rollup'))
15+
16+
clusterModules project(xpackModule('downsample'))
17+
clusterModules project(xpackModule('analytics'))
18+
clusterModules project(xpackModule('ilm'))
19+
clusterModules project(':modules:lang-painless')
20+
clusterModules project(':modules:mapper-extras')
21+
clusterModules project(':modules:data-streams')
22+
clusterModules project(':modules:aggregations')
23+
clusterModules project(':modules:ingest-common')
1524
}
1625

1726
restResources {
@@ -24,12 +33,6 @@ artifacts {
2433
restXpackTests(new File(projectDir, "src/yamlRestTest/resources/rest-api-spec/test"))
2534
}
2635

27-
tasks.named('yamlRestTest') {
28-
usesDefaultDistribution("to be triaged")
29-
}
30-
tasks.named('yamlRestCompatTest') {
31-
usesDefaultDistribution("to be triaged")
32-
}
3336
if (buildParams.inFipsJvm){
3437
// This test cluster is using a BASIC license and FIPS 140 mode is not supported in BASIC
3538
tasks.named("yamlRestTest").configure{enabled = false }

x-pack/plugin/downsample/qa/rest/src/yamlRestTest/java/org/elasticsearch/xpack/downsample/DownsampleRestIT.java

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@
1010
import com.carrotsearch.randomizedtesting.annotations.ParametersFactory;
1111

1212
import org.elasticsearch.test.cluster.ElasticsearchCluster;
13-
import org.elasticsearch.test.cluster.local.distribution.DistributionType;
1413
import org.elasticsearch.test.rest.yaml.ClientYamlTestCandidate;
1514
import org.elasticsearch.test.rest.yaml.ESClientYamlSuiteTestCase;
1615
import org.junit.ClassRule;
@@ -19,7 +18,14 @@ public class DownsampleRestIT extends ESClientYamlSuiteTestCase {
1918

2019
@ClassRule
2120
public static ElasticsearchCluster cluster = ElasticsearchCluster.local()
22-
.distribution(DistributionType.DEFAULT)
21+
.module("x-pack-downsample")
22+
.module("x-pack-ilm")
23+
.module("lang-painless")
24+
.module("aggregations") // for auto_date_histogram
25+
.module("mapper-extras") // for scaled_float
26+
.module("x-pack-analytics") // for histogram
27+
.module("data-streams") // for time series
28+
.module("ingest-common")
2329
.setting("xpack.license.self_generated.type", "trial")
2430
.setting("xpack.security.enabled", "false")
2531
.build();

x-pack/plugin/downsample/qa/rest/src/yamlRestTest/java/org/elasticsearch/xpack/downsample/DownsampleWithBasicRestIT.java

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@
1010
import com.carrotsearch.randomizedtesting.annotations.ParametersFactory;
1111

1212
import org.elasticsearch.test.cluster.ElasticsearchCluster;
13-
import org.elasticsearch.test.cluster.local.distribution.DistributionType;
1413
import org.elasticsearch.test.rest.yaml.ClientYamlTestCandidate;
1514
import org.elasticsearch.test.rest.yaml.ESClientYamlSuiteTestCase;
1615
import org.junit.ClassRule;
@@ -19,7 +18,14 @@ public class DownsampleWithBasicRestIT extends ESClientYamlSuiteTestCase {
1918

2019
@ClassRule
2120
public static ElasticsearchCluster cluster = ElasticsearchCluster.local()
22-
.distribution(DistributionType.DEFAULT)
21+
.module("x-pack-downsample")
22+
.module("x-pack-ilm")
23+
.module("lang-painless")
24+
.module("aggregations") // for auto_date_histogram
25+
.module("mapper-extras") // for scaled_float
26+
.module("x-pack-analytics") // for histogram
27+
.module("data-streams") // for time series
28+
.module("ingest-common")
2329
.setting("xpack.security.enabled", "false")
2430
.build();
2531

x-pack/plugin/downsample/qa/rest/src/yamlRestTest/java/org/elasticsearch/xpack/downsample/DownsampleWithSecurityRestIT.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@
1313
import org.elasticsearch.common.settings.Settings;
1414
import org.elasticsearch.common.util.concurrent.ThreadContext;
1515
import org.elasticsearch.test.cluster.ElasticsearchCluster;
16-
import org.elasticsearch.test.cluster.local.distribution.DistributionType;
1716
import org.elasticsearch.test.rest.yaml.ClientYamlTestCandidate;
1817
import org.elasticsearch.test.rest.yaml.ESClientYamlSuiteTestCase;
1918
import org.junit.ClassRule;
@@ -25,7 +24,8 @@ public class DownsampleWithSecurityRestIT extends ESClientYamlSuiteTestCase {
2524

2625
@ClassRule
2726
public static ElasticsearchCluster cluster = ElasticsearchCluster.local()
28-
.distribution(DistributionType.DEFAULT)
27+
.module("x-pack-downsample")
28+
.module("x-pack-analytics") // for histogram
2929
.setting("xpack.license.self_generated.type", "trial")
3030
.setting("xpack.security.enabled", "true")
3131
.user(USERNAME, PASSWORD)

0 commit comments

Comments
 (0)