Skip to content

Commit 02cd4b6

Browse files
committed
Update version skips and constants after backport
After elastic#41906 was backported, we need to update the various test skips and version constants
1 parent e126d18 commit 02cd4b6

File tree

9 files changed

+15
-14
lines changed

9 files changed

+15
-14
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 & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -109,6 +109,7 @@
109109
- do:
110110
warnings:
111111
- '[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.'
112113
ml.put_datafeed:
113114
datafeed_id: mixed-cluster-datafeed-with-aggs
114115
body: >

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,8 @@
4848
---
4949
"Put job and datafeed with aggs in old cluster - pre-deprecated interval":
5050
- skip:
51-
version: "all" #TODO change this after backport
52-
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"
5353

5454
- do:
5555
ml.put_job:
@@ -118,8 +118,8 @@
118118
---
119119
"Put job and datafeed with aggs in old cluster - deprecated interval with warning":
120120
- skip:
121-
version: " - 7.99.99" #TODO change this after backport
122-
reason: calendar_interval introduced in 7.1.0
121+
version: " - 7.1.99"
122+
reason: calendar_interval introduced in 7.2.0
123123
features: warnings
124124

125125
- do:

0 commit comments

Comments
 (0)