Skip to content

Commit 385dfd9

Browse files
authored
Update version skips and constants after backport (#42290)
After #41906 was backported, we need to update the various test skips and version constants
1 parent 28aae64 commit 385dfd9

File tree

10 files changed

+15
-28
lines changed

10 files changed

+15
-28
lines changed

rest-api-spec/src/main/resources/rest-api-spec/test/search.aggregation/230_composite.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -241,7 +241,7 @@ setup:
241241
---
242242
"Composite aggregation with format":
243243
- skip:
244-
version: " - 7.99.99" #TODO change this after backport
244+
version: " - 7.1.99"
245245
reason: calendar_interval introduced in 7.2.0
246246
features: warnings
247247

@@ -307,7 +307,7 @@ setup:
307307
---
308308
"Composite aggregation with format and calendar_interval":
309309
- skip:
310-
version: " - 7.99.99" #TODO change this after backport
310+
version: " - 7.1.99"
311311
reason: calendar_interval introduced in 7.2.0
312312

313313
- do:

rest-api-spec/src/main/resources/rest-api-spec/test/search.aggregation/250_moving_fn.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
"Bad window":
33

44
- skip:
5-
version: " - 7.99.0" #TODO change this after backport
5+
version: " - 7.1.99"
66
reason: "calendar_interval added in 7.2"
77

88
- do:
@@ -30,7 +30,7 @@
3030
"Bad window deprecated interval":
3131

3232
- skip:
33-
version: " - 7.99.0" #TODO change this after backport
33+
version: " - 7.1.99"
3434
reason: "interval deprecation added in 7.2"
3535
features: "warnings"
3636

rest-api-spec/src/main/resources/rest-api-spec/test/search.aggregation/80_typed_keys.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,7 @@ setup:
206206
---
207207
"Test typed keys parameter for date_histogram aggregation and max_bucket pipeline aggregation":
208208
- skip:
209-
version: " - 7.99.0" #TODO change this after backport
209+
version: " - 7.1.99"
210210
reason: "calendar_interval added in 7.2"
211211
- do:
212212
search:

rest-api-spec/src/main/resources/rest-api-spec/test/search/240_date_nanos.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ setup:
123123
---
124124
"date histogram aggregation with date and date_nanos mapping":
125125
- skip:
126-
version: " - 7.99.99" #TODO change this after backport
126+
version: " - 7.1.99"
127127
reason: calendar_interval introduced in 7.2.0
128128

129129
- do:

server/src/main/java/org/elasticsearch/search/aggregations/bucket/histogram/DateIntervalWrapper.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ public static <T extends DateIntervalConsumer> void declareIntervalFields(Object
113113
public DateIntervalWrapper() {}
114114

115115
public DateIntervalWrapper(StreamInput in) throws IOException {
116-
if (in.getVersion().before(Version.V_8_0_0)) { // TODO change this after backport
116+
if (in.getVersion().before(Version.V_7_2_0)) {
117117
long interval = in.readLong();
118118
DateHistogramInterval histoInterval = in.readOptionalWriteable(DateHistogramInterval::new);
119119

@@ -374,7 +374,7 @@ public boolean isEmpty() {
374374

375375
@Override
376376
public void writeTo(StreamOutput out) throws IOException {
377-
if (out.getVersion().before(Version.V_8_0_0)) { // TODO change this after backport
377+
if (out.getVersion().before(Version.V_7_2_0)) {
378378
if (intervalType.equals(IntervalTypeEnum.LEGACY_INTERVAL)) {
379379
out.writeLong(TimeValue.parseTimeValue(dateHistogramInterval.toString(),
380380
DateHistogramAggregationBuilder.NAME + ".innerWriteTo").getMillis());

x-pack/qa/full-cluster-restart/src/test/java/org/elasticsearch/xpack/restart/FullClusterRestartIT.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -229,7 +229,7 @@ public void testRollupAfterRestart() throws Exception {
229229
final Request createRollupJobRequest = new Request("PUT", getRollupEndpoint() + "/job/rollup-job-test");
230230

231231
String intervalType;
232-
if (getOldClusterVersion().onOrAfter(Version.V_8_0_0)) { // TODO change this after backport
232+
if (getOldClusterVersion().onOrAfter(Version.V_7_2_0)) {
233233
intervalType = "fixed_interval";
234234
} else {
235235
intervalType = "interval";

x-pack/qa/rolling-upgrade/src/test/java/org/elasticsearch/upgrades/RollupDateHistoUpgradeIT.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ public class RollupDateHistoUpgradeIT extends AbstractUpgradeTestCase {
3434
Version.fromString(System.getProperty("tests.upgrade_from_version"));
3535

3636
public void testDateHistoIntervalUpgrade() throws Exception {
37-
assumeTrue("DateHisto interval changed in 7.1", UPGRADE_FROM_VERSION.before(Version.V_7_2_0));
37+
assumeTrue("DateHisto interval changed in 7.2", UPGRADE_FROM_VERSION.before(Version.V_7_2_0));
3838
switch (CLUSTER_TYPE) {
3939
case OLD:
4040
break;

x-pack/qa/rolling-upgrade/src/test/resources/rest-api-spec/test/mixed_cluster/40_ml_datafeed_crud.yml

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,3 @@
1-
setup:
2-
- skip:
3-
version: "all"
4-
reason: "AwaitsFix https://github.com/elastic/elasticsearch/issues/42258"
5-
61
---
72
"Test old cluster datafeed without aggs":
83
- do:
@@ -114,6 +109,7 @@ setup:
114109
- do:
115110
warnings:
116111
- '[interval] on [date_histogram] is deprecated, use [fixed_interval] or [calendar_interval] in the future.'
112+
- '[interval] on [date_histogram] is deprecated, use [fixed_interval] or [calendar_interval] in the future.'
117113
ml.put_datafeed:
118114
datafeed_id: mixed-cluster-datafeed-with-aggs
119115
body: >

x-pack/qa/rolling-upgrade/src/test/resources/rest-api-spec/test/old_cluster/40_ml_datafeed_crud.yml

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,3 @@
1-
setup:
2-
- skip:
3-
version: "all"
4-
reason: "AwaitsFix https://github.com/elastic/elasticsearch/issues/42258"
5-
61
---
72
"Put job and datafeed without aggs in old cluster":
83

@@ -53,8 +48,8 @@ setup:
5348
---
5449
"Put job and datafeed with aggs in old cluster - pre-deprecated interval":
5550
- skip:
56-
version: "all" #TODO change this after backport
57-
reason: "AwaitsFix https://github.com/elastic/elasticsearch/issues/42258; calendar_interval introduced in 7.2.0"
51+
version: "7.1.99 - "
52+
reason: "calendar_interval introduced in 7.2.0"
5853

5954
- do:
6055
ml.put_job:
@@ -123,8 +118,8 @@ setup:
123118
---
124119
"Put job and datafeed with aggs in old cluster - deprecated interval with warning":
125120
- skip:
126-
version: " - 7.99.99" #TODO change this after backport
127-
reason: calendar_interval introduced in 7.1.0
121+
version: " - 7.1.99"
122+
reason: calendar_interval introduced in 7.2.0
128123
features: warnings
129124

130125
- do:

x-pack/qa/rolling-upgrade/src/test/resources/rest-api-spec/test/upgraded_cluster/40_ml_datafeed_crud.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,4 @@
11
setup:
2-
- skip:
3-
version: "all"
4-
reason: "AwaitsFix https://github.com/elastic/elasticsearch/issues/42258"
5-
62
- do:
73
cluster.health:
84
wait_for_status: green

0 commit comments

Comments
 (0)