Skip to content

Commit 844f350

Browse files
author
Hendrik Muhs
committed
[Transform] restructure transform yaml tests (#52956)
restructure transform yaml tests to run cleanup in teardown phase relates #52428
1 parent d9258e2 commit 844f350

File tree

4 files changed

+178
-160
lines changed

4 files changed

+178
-160
lines changed

x-pack/plugin/src/test/resources/rest-api-spec/test/transform/transforms_crud.yml

Lines changed: 0 additions & 43 deletions
Original file line numberDiff line numberDiff line change
@@ -605,49 +605,6 @@ setup:
605605
"version": "7.3.0"
606606
}
607607
---
608-
"Test force deleting a running transform":
609-
- do:
610-
transform.put_transform:
611-
transform_id: "airline-transform-start-delete"
612-
body: >
613-
{
614-
"source": { "index": "airline-data" },
615-
"dest": { "index": "airline-data-by-airline-start-delete" },
616-
"pivot": {
617-
"group_by": { "airline": {"terms": {"field": "airline"}}},
618-
"aggs": {"avg_response": {"avg": {"field": "responsetime"}}}
619-
},
620-
"sync": {
621-
"time": {
622-
"field": "time",
623-
"delay": "90m"
624-
}
625-
}
626-
}
627-
- match: { acknowledged: true }
628-
- do:
629-
transform.start_transform:
630-
transform_id: "airline-transform-start-delete"
631-
- match: { acknowledged: true }
632-
633-
- do:
634-
transform.get_transform_stats:
635-
transform_id: "airline-transform-start-delete"
636-
- match: { count: 1 }
637-
- match: { transforms.0.id: "airline-transform-start-delete" }
638-
- match: { transforms.0.state: "/started|indexing/" }
639-
640-
- do:
641-
catch: /Cannot delete transform \[airline-transform-start-delete\] as the task is running/
642-
transform.delete_transform:
643-
transform_id: "airline-transform-start-delete"
644-
645-
- do:
646-
transform.delete_transform:
647-
transform_id: "airline-transform-start-delete"
648-
force: true
649-
- match: { acknowledged: true }
650-
---
651608
"Test put transform with missing pipeline":
652609
- do:
653610
catch: /Pipeline with id \[missing-transform-pipeline\] could not be found/
Lines changed: 70 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,70 @@
1+
setup:
2+
- do:
3+
indices.create:
4+
index: airline-data
5+
body:
6+
mappings:
7+
properties:
8+
time:
9+
type: date
10+
airline:
11+
type: keyword
12+
responsetime:
13+
type: float
14+
event_rate:
15+
type: integer
16+
- do:
17+
transform.put_transform:
18+
transform_id: "airline-transform-start-delete"
19+
body: >
20+
{
21+
"source": { "index": "airline-data" },
22+
"dest": { "index": "airline-data-by-airline-start-delete" },
23+
"pivot": {
24+
"group_by": { "airline": {"terms": {"field": "airline"}}},
25+
"aggs": {"avg_response": {"avg": {"field": "responsetime"}}}
26+
},
27+
"sync": {
28+
"time": {
29+
"field": "time",
30+
"delay": "90m"
31+
}
32+
}
33+
}
34+
35+
---
36+
teardown:
37+
- do:
38+
transform.stop_transform:
39+
wait_for_checkpoint: false
40+
transform_id: "airline-transform-start-delete"
41+
wait_for_completion: true
42+
ignore: 404
43+
- do:
44+
transform.delete_transform:
45+
transform_id: "airline-transform-start-delete"
46+
ignore: 404
47+
---
48+
"Test force deleting a running transform":
49+
- do:
50+
transform.start_transform:
51+
transform_id: "airline-transform-start-delete"
52+
- match: { acknowledged: true }
53+
54+
- do:
55+
transform.get_transform_stats:
56+
transform_id: "airline-transform-start-delete"
57+
- match: { count: 1 }
58+
- match: { transforms.0.id: "airline-transform-start-delete" }
59+
- match: { transforms.0.state: "/started|indexing/" }
60+
61+
- do:
62+
catch: /Cannot delete transform \[airline-transform-start-delete\] as the task is running/
63+
transform.delete_transform:
64+
transform_id: "airline-transform-start-delete"
65+
66+
- do:
67+
transform.delete_transform:
68+
transform_id: "airline-transform-start-delete"
69+
force: true
70+
- match: { acknowledged: true }

x-pack/plugin/src/test/resources/rest-api-spec/test/transform/transforms_stats.yml

Lines changed: 34 additions & 117 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,30 @@ setup:
2525
"aggs": {"avg_response": {"avg": {"field": "responsetime"}}}
2626
}
2727
}
28+
- do:
29+
transform.put_transform:
30+
transform_id: "airline-transform-stats-dos"
31+
body: >
32+
{
33+
"source": { "index": "airline-data" },
34+
"dest": { "index": "airline-data-by-airline-stats-dos" },
35+
"pivot": {
36+
"group_by": { "airline": {"terms": {"field": "airline"}}},
37+
"aggs": {"avg_response": {"avg": {"field": "responsetime"}}}
38+
}
39+
}
40+
- do:
41+
transform.put_transform:
42+
transform_id: "airline-transform-stats-the-third"
43+
body: >
44+
{
45+
"source": { "index": "airline-data" },
46+
"dest": { "index": "airline-data-by-airline-stats-the-third" },
47+
"pivot": {
48+
"group_by": { "airline": {"terms": {"field": "airline"}}},
49+
"aggs": {"avg_response": {"avg": {"field": "responsetime"}}}
50+
}
51+
}
2852
- do:
2953
transform.start_transform:
3054
transform_id: "airline-transform-stats"
@@ -36,11 +60,16 @@ teardown:
3660
wait_for_checkpoint: false
3761
transform_id: "airline-transform-stats"
3862
wait_for_completion: true
39-
4063
- do:
4164
transform.delete_transform:
4265
transform_id: "airline-transform-stats"
66+
- do:
67+
transform.delete_transform:
68+
transform_id: "airline-transform-stats-dos"
4369

70+
- do:
71+
transform.delete_transform:
72+
transform_id: "airline-transform-stats-the-third"
4473
---
4574
"Test get transform stats":
4675
- do:
@@ -71,30 +100,6 @@ teardown:
71100

72101
---
73102
"Test get multiple transform stats":
74-
- do:
75-
transform.put_transform:
76-
transform_id: "airline-transform-stats-dos"
77-
body: >
78-
{
79-
"source": { "index": "airline-data" },
80-
"dest": { "index": "airline-data-by-airline-stats-dos" },
81-
"pivot": {
82-
"group_by": { "airline": {"terms": {"field": "airline"}}},
83-
"aggs": {"avg_response": {"avg": {"field": "responsetime"}}}
84-
}
85-
}
86-
- do:
87-
transform.put_transform:
88-
transform_id: "airline-transform-stats-the-third"
89-
body: >
90-
{
91-
"source": { "index": "airline-data" },
92-
"dest": { "index": "airline-data-by-airline-stats-the-third" },
93-
"pivot": {
94-
"group_by": { "airline": {"terms": {"field": "airline"}}},
95-
"aggs": {"avg_response": {"avg": {"field": "responsetime"}}}
96-
}
97-
}
98103
- do:
99104
transform.get_transform_stats:
100105
transform_id: "*"
@@ -135,58 +140,26 @@ teardown:
135140
- match: { transforms.0.id: "airline-transform-stats-dos" }
136141
- match: { transforms.1.id: "airline-transform-stats-the-third" }
137142

138-
- do:
139-
transform.delete_transform:
140-
transform_id: "airline-transform-stats-dos"
141-
142-
- do:
143-
transform.delete_transform:
144-
transform_id: "airline-transform-stats-the-third"
145-
146-
147143
---
148144
"Test get multiple transform stats where one does not have a task":
149-
- do:
150-
transform.put_transform:
151-
transform_id: "airline-transform-stats-dos"
152-
body: >
153-
{
154-
"source": { "index": "airline-data" },
155-
"dest": { "index": "airline-data-by-airline-stats-dos" },
156-
"pivot": {
157-
"group_by": { "airline": {"terms": {"field": "airline"}}},
158-
"aggs": {"avg_response": {"avg": {"field": "responsetime"}}}
159-
}
160-
}
161145
- do:
162146
transform.get_transform_stats:
163147
transform_id: "*"
164-
- match: { count: 2 }
148+
- match: { count: 3 }
165149
- match: { transforms.0.id: "airline-transform-stats" }
166150
- match: { transforms.1.id: "airline-transform-stats-dos" }
151+
- match: { transforms.2.id: "airline-transform-stats-the-third" }
167152

168153
- do:
169154
transform.get_transform_stats:
170155
transform_id: "_all"
171-
- match: { count: 2 }
156+
- match: { count: 3 }
172157
- match: { transforms.0.id: "airline-transform-stats" }
173158
- match: { transforms.1.id: "airline-transform-stats-dos" }
159+
- match: { transforms.2.id: "airline-transform-stats-the-third" }
174160

175161
---
176162
"Test get single transform stats when it does not have a task":
177-
178-
- do:
179-
transform.put_transform:
180-
transform_id: "airline-transform-stats-dos"
181-
body: >
182-
{
183-
"source": { "index": "airline-data" },
184-
"dest": { "index": "airline-data-by-airline-stats-dos" },
185-
"pivot": {
186-
"group_by": { "airline": {"terms": {"field": "airline"}}},
187-
"aggs": {"avg_response": {"avg": {"field": "responsetime"}}}
188-
}
189-
}
190163
- do:
191164
transform.get_transform_stats:
192165
transform_id: "airline-transform-stats-dos"
@@ -204,59 +177,3 @@ teardown:
204177
- match: { transforms.0.stats.search_time_in_ms: 0 }
205178
- match: { transforms.0.stats.search_total: 0 }
206179
- match: { transforms.0.stats.search_failures: 0 }
207-
---
208-
"Test get continuous transform stats":
209-
- do:
210-
transform.put_transform:
211-
transform_id: "airline-transform-stats-continuous"
212-
body: >
213-
{
214-
"source": { "index": "airline-data" },
215-
"dest": { "index": "airline-data-by-airline-stats-continuous" },
216-
"pivot": {
217-
"group_by": { "airline": {"terms": {"field": "airline"}}},
218-
"aggs": {"avg_response": {"avg": {"field": "responsetime"}}}
219-
},
220-
"sync": { "time": { "field": "time", "delay": "1m" } }
221-
}
222-
- do:
223-
transform.start_transform:
224-
transform_id: "airline-transform-stats-continuous"
225-
- do:
226-
transform.get_transform_stats:
227-
transform_id: "airline-transform-stats-continuous"
228-
- match: { count: 1 }
229-
- match: { transforms.0.id: "airline-transform-stats-continuous" }
230-
# Since this is continuous, there is no worry of it automatically stopping
231-
- match: { transforms.0.state: "/started|indexing/" }
232-
- lte: { transforms.0.checkpointing.last.checkpoint: 1 }
233-
# Since this is continuous, and _start does not return until it is assigned
234-
# we should see a node assignment
235-
- is_true: transforms.0.node
236-
- is_true: transforms.0.node.id
237-
- is_true: transforms.0.node.name
238-
- is_true: transforms.0.node.ephemeral_id
239-
- is_true: transforms.0.node.transport_address
240-
- lte: { transforms.0.stats.pages_processed: 1 }
241-
- match: { transforms.0.stats.documents_processed: 0 }
242-
- match: { transforms.0.stats.documents_indexed: 0 }
243-
- lte: { transforms.0.stats.trigger_count: 1 }
244-
- match: { transforms.0.stats.index_time_in_ms: 0 }
245-
- match: { transforms.0.stats.index_total: 0 }
246-
- match: { transforms.0.stats.index_failures: 0 }
247-
- gte: { transforms.0.stats.search_time_in_ms: 0 }
248-
- lte: { transforms.0.stats.search_total: 1 }
249-
- match: { transforms.0.stats.search_failures: 0 }
250-
- is_true: transforms.0.stats.exponential_avg_checkpoint_duration_ms
251-
- match: { transforms.0.stats.exponential_avg_documents_indexed: 0.0 }
252-
- match: { transforms.0.stats.exponential_avg_documents_processed: 0.0 }
253-
254-
- do:
255-
transform.stop_transform:
256-
wait_for_checkpoint: false
257-
transform_id: "airline-transform-stats-continuous"
258-
wait_for_completion: true
259-
260-
- do:
261-
transform.delete_transform:
262-
transform_id: "airline-transform-stats-continuous"
Lines changed: 74 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,74 @@
1+
setup:
2+
- do:
3+
indices.create:
4+
index: airline-data
5+
body:
6+
mappings:
7+
properties:
8+
time:
9+
type: date
10+
airline:
11+
type: keyword
12+
responsetime:
13+
type: float
14+
event_rate:
15+
type: integer
16+
- do:
17+
transform.put_transform:
18+
transform_id: "airline-transform-stats-continuous"
19+
body: >
20+
{
21+
"source": { "index": "airline-data" },
22+
"dest": { "index": "airline-data-by-airline-stats-continuous" },
23+
"pivot": {
24+
"group_by": { "airline": {"terms": {"field": "airline"}}},
25+
"aggs": {"avg_response": {"avg": {"field": "responsetime"}}}
26+
},
27+
"sync": { "time": { "field": "time", "delay": "1m" } }
28+
}
29+
- do:
30+
transform.start_transform:
31+
transform_id: "airline-transform-stats-continuous"
32+
33+
---
34+
teardown:
35+
- do:
36+
transform.stop_transform:
37+
wait_for_checkpoint: false
38+
transform_id: "airline-transform-stats-continuous"
39+
wait_for_completion: true
40+
41+
- do:
42+
transform.delete_transform:
43+
transform_id: "airline-transform-stats-continuous"
44+
45+
---
46+
"Test get continuous transform stats":
47+
- do:
48+
transform.get_transform_stats:
49+
transform_id: "airline-transform-stats-continuous"
50+
- match: { count: 1 }
51+
- match: { transforms.0.id: "airline-transform-stats-continuous" }
52+
# Since this is continuous, there is no worry of it automatically stopping
53+
- match: { transforms.0.state: "/started|indexing/" }
54+
- lte: { transforms.0.checkpointing.last.checkpoint: 1 }
55+
# Since this is continuous, and _start does not return until it is assigned
56+
# we should see a node assignment
57+
- is_true: transforms.0.node
58+
- is_true: transforms.0.node.id
59+
- is_true: transforms.0.node.name
60+
- is_true: transforms.0.node.ephemeral_id
61+
- is_true: transforms.0.node.transport_address
62+
- lte: { transforms.0.stats.pages_processed: 1 }
63+
- match: { transforms.0.stats.documents_processed: 0 }
64+
- match: { transforms.0.stats.documents_indexed: 0 }
65+
- lte: { transforms.0.stats.trigger_count: 1 }
66+
- match: { transforms.0.stats.index_time_in_ms: 0 }
67+
- match: { transforms.0.stats.index_total: 0 }
68+
- match: { transforms.0.stats.index_failures: 0 }
69+
- gte: { transforms.0.stats.search_time_in_ms: 0 }
70+
- lte: { transforms.0.stats.search_total: 1 }
71+
- match: { transforms.0.stats.search_failures: 0 }
72+
- is_true: transforms.0.stats.exponential_avg_checkpoint_duration_ms
73+
- match: { transforms.0.stats.exponential_avg_documents_indexed: 0.0 }
74+
- match: { transforms.0.stats.exponential_avg_documents_processed: 0.0 }

0 commit comments

Comments
 (0)