1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
|
# This Source Code Form is subject to the terms of the Mozilla Public
# License, v. 2.0. If a copy of the MPL was not distributed with this
# file, You can obtain one at http://mozilla.org/MPL/2.0/.
from collections import defaultdict
from fnmatch import fnmatch
from taskgraph.optimize.base import OptimizationStrategy, register_strategy, registry
from gecko_taskgraph.util.bugbug import (
CT_HIGH,
CT_LOW,
CT_MEDIUM,
BugbugTimeoutException,
push_schedules,
)
from gecko_taskgraph.util.hg import get_push_data
FALLBACK = "skip-unless-has-relevant-tests"
def merge_bugbug_replies(data, new_data):
"""Merge a bugbug reply (stored in the `new_data` argument) into another (stored
in the `data` argument).
"""
for key, value in new_data.items():
if isinstance(value, dict):
if key not in data:
data[key] = {}
if len(value) == 0:
continue
dict_value = next(iter(value.values()))
if isinstance(dict_value, list):
for name, configs in value.items():
if name not in data[key]:
data[key][name] = set()
data[key][name].update(configs)
else:
for name, confidence in value.items():
if name not in data[key] or data[key][name] < confidence:
data[key][name] = confidence
elif isinstance(value, list):
if key not in data:
data[key] = set()
data[key].update(value)
@register_strategy("bugbug-low", args=(CT_LOW,))
@register_strategy("bugbug-medium", args=(CT_MEDIUM,))
@register_strategy("bugbug-high", args=(CT_HIGH,))
@register_strategy("bugbug-tasks-medium", args=(CT_MEDIUM, True))
@register_strategy("bugbug-tasks-high", args=(CT_HIGH, True))
@register_strategy("bugbug-reduced", args=(CT_MEDIUM, True, True))
@register_strategy("bugbug-reduced-fallback", args=(CT_MEDIUM, True, True, FALLBACK))
@register_strategy("bugbug-reduced-high", args=(CT_HIGH, True, True))
@register_strategy("bugbug-reduced-manifests", args=(CT_MEDIUM, False, True))
@register_strategy(
"bugbug-reduced-manifests-config-selection-low",
args=(CT_LOW, False, True, None, 1, True),
)
@register_strategy(
"bugbug-reduced-manifests-config-selection",
args=(CT_MEDIUM, False, True, None, 1, True),
)
@register_strategy(
"bugbug-reduced-manifests-fallback-low", args=(CT_LOW, False, True, FALLBACK)
)
@register_strategy(
"bugbug-reduced-manifests-fallback", args=(CT_MEDIUM, False, True, FALLBACK)
)
@register_strategy(
"bugbug-reduced-manifests-fallback-last-10-pushes",
args=(0.3, False, True, FALLBACK, 10),
)
class BugBugPushSchedules(OptimizationStrategy):
"""Query the 'bugbug' service to retrieve relevant tasks and manifests.
Args:
confidence_threshold (float): The minimum confidence threshold (in
range [0, 1]) needed for a task to be scheduled.
tasks_only (bool): Whether or not to only use tasks and no groups
(default: False)
use_reduced_tasks (bool): Whether or not to use the reduced set of tasks
provided by the bugbug service (default: False).
fallback (str): The fallback strategy to use if there
was a failure in bugbug (default: None)
num_pushes (int): The number of pushes to consider for the selection
(default: 1).
select_configs (bool): Whether to select configurations for manifests
too (default: False).
"""
def __init__(
self,
confidence_threshold,
tasks_only=False,
use_reduced_tasks=False,
fallback=None,
num_pushes=1,
select_configs=False,
):
self.confidence_threshold = confidence_threshold
self.use_reduced_tasks = use_reduced_tasks
self.fallback = fallback
self.tasks_only = tasks_only
self.num_pushes = num_pushes
self.select_configs = select_configs
self.timedout = False
def should_remove_task(self, task, params, importance):
project = params["project"]
if project not in ("autoland", "try"):
return False
current_push_id = int(params["pushlog_id"])
rev = params["head_rev"]
if self.timedout:
return registry[self.fallback].should_remove_task(task, params, importance)
data = {}
start_push_id = current_push_id - self.num_pushes + 1
if self.num_pushes != 1:
push_data = get_push_data(
params["head_repository"], project, start_push_id, current_push_id - 1
)
for push_id in range(start_push_id, current_push_id + 1):
if push_id == current_push_id:
rev = params["head_rev"]
else:
rev = push_data[push_id]["changesets"][-1]
try:
new_data = push_schedules(params["project"], rev)
merge_bugbug_replies(data, new_data)
except BugbugTimeoutException:
if not self.fallback:
raise
self.timedout = True
return self.should_remove_task(task, params, importance)
key = "reduced_tasks" if self.use_reduced_tasks else "tasks"
tasks = {
task
for task, confidence in data.get(key, {}).items()
if confidence >= self.confidence_threshold
}
test_manifests = task.attributes.get("test_manifests")
if test_manifests is None or self.tasks_only:
if data.get("known_tasks") and task.label not in data["known_tasks"]:
return False
if task.label not in tasks:
return True
return False
# If a task contains more than one group, use the max confidence.
groups = data.get("groups", {})
confidences = [c for g, c in groups.items() if g in test_manifests]
if not confidences or max(confidences) < self.confidence_threshold:
return True
# If the task configuration doesn't match the ones selected by bugbug for
# the manifests, optimize out.
if self.select_configs:
selected_groups = [
g
for g, c in groups.items()
if g in test_manifests and c > self.confidence_threshold
]
config_groups = data.get("config_groups", defaultdict(list))
# Configurations returned by bugbug are in a format such as
# `test-windows10-64/opt-*-e10s`, while task labels are like
# test-windows10-64-qr/opt-mochitest-browser-chrome-e10s-6.
# In order to match the strings, we need to ignore the chunk number
# from the task label.
parts = task.label.split("-")
label_without_chunk_number = "-".join(
parts[:-1] if parts[-1].isdigit() else parts
)
if not any(
fnmatch(label_without_chunk_number, config)
for group in selected_groups
for config in config_groups[group]
):
return True
# Store group importance so future optimizers can access it.
for manifest in test_manifests:
if manifest not in groups:
continue
confidence = groups[manifest]
if confidence >= CT_HIGH:
importance[manifest] = "high"
elif confidence >= CT_MEDIUM:
importance[manifest] = "medium"
elif confidence >= CT_LOW:
importance[manifest] = "low"
else:
importance[manifest] = "lowest"
return False
@register_strategy("platform-debug")
class SkipUnlessDebug(OptimizationStrategy):
"""Only run debug platforms."""
def should_remove_task(self, task, params, arg):
return (
"build_type" in task.attributes and task.attributes["build_type"] != "debug"
)
@register_strategy("platform-disperse")
@register_strategy("platform-disperse-no-unseen", args=(None, 0))
@register_strategy(
"platform-disperse-only-one",
args=(
{
"high": 1,
"medium": 1,
"low": 1,
"lowest": 0,
},
0,
),
)
class DisperseGroups(OptimizationStrategy):
"""Disperse groups across test configs.
Each task has an associated 'importance' dict passed in via the arg. This
is of the form `{<group>: <importance>}`.
Where 'group' is a test group id (usually a path to a manifest), and 'importance' is
one of `{'lowest', 'low', 'medium', 'high'}`.
Each importance value has an associated 'count' as defined in
`self.target_counts`. It guarantees that 'manifest' will run in at least
'count' different configurations (assuming there are enough tasks
containing 'manifest').
On configurations that haven't been seen before, we'll increase the target
count by `self.unseen_modifier` to increase the likelihood of scheduling a
task on that configuration.
Args:
target_counts (dict): Override DEFAULT_TARGET_COUNTS with custom counts. This
is a dict mapping the importance value ('lowest', 'low', etc) to the
minimum number of configurations manifests with this value should run
on.
unseen_modifier (int): Override DEFAULT_UNSEEN_MODIFIER to a custom
value. This is the amount we'll increase 'target_count' by for unseen
configurations.
"""
DEFAULT_TARGET_COUNTS = {
"high": 3,
"medium": 2,
"low": 1,
"lowest": 0,
}
DEFAULT_UNSEEN_MODIFIER = 1
def __init__(self, target_counts=None, unseen_modifier=DEFAULT_UNSEEN_MODIFIER):
self.target_counts = self.DEFAULT_TARGET_COUNTS.copy()
if target_counts:
self.target_counts.update(target_counts)
self.unseen_modifier = unseen_modifier
self.count = defaultdict(int)
self.seen_configurations = set()
def should_remove_task(self, task, params, importance):
test_manifests = task.attributes.get("test_manifests")
test_platform = task.attributes.get("test_platform")
if not importance or not test_manifests or not test_platform:
return False
# Build the test configuration key.
key = test_platform
if "unittest_variant" in task.attributes:
key += "-" + task.attributes["unittest_variant"]
important_manifests = set(test_manifests) & set(importance)
for manifest in important_manifests:
target_count = self.target_counts[importance[manifest]]
# If this configuration hasn't been seen before, increase the
# likelihood of scheduling the task.
if key not in self.seen_configurations:
target_count += self.unseen_modifier
if self.count[manifest] < target_count:
# Update manifest counts and seen configurations.
self.seen_configurations.add(key)
for manifest in important_manifests:
self.count[manifest] += 1
return False
# Should remove task because all manifests have reached their
# importance count (or there were no important manifests).
return True
|