Skip to content

Commit

Permalink
Disable parallelization on telemetry_unittests & set the number of sh…
Browse files Browse the repository at this point in the history
…ards to 2 (linux)

It's a known problem that launching browsers in parallel on Linux will cause the
test to hang due to deadlock (
catapult-project/catapult#3074 (comment)),
and since each hang test takes 1 min timeout, so we disable telemetry
parallelization & shards it on 2 machines to reduce the overall wait time.

BUG=676742

Review-Url: https://codereview.chromium.org/2653163002
Cr-Commit-Position: refs/heads/master@{#445998}
  • Loading branch information
nedn authored and Commit bot committed Jan 25, 2017
1 parent 68693ab commit 027b82c
Show file tree
Hide file tree
Showing 2 changed files with 25 additions and 5 deletions.
12 changes: 10 additions & 2 deletions testing/buildbot/chromium.fyi.json
Original file line number Diff line number Diff line change
Expand Up @@ -10562,13 +10562,17 @@
}
},
{
"args": [
"--jobs=1"
],
"isolate_name": "telemetry_unittests",
"name": "telemetry_unittests",
"override_compile_targets": [
"telemetry_unittests_run"
],
"swarming": {
"can_use_on_swarming_builders": true
"can_use_on_swarming_builders": true,
"shards": 2
}
}
],
Expand Down Expand Up @@ -11127,13 +11131,17 @@
}
},
{
"args": [
"--jobs=1"
],
"isolate_name": "telemetry_unittests",
"name": "telemetry_unittests",
"override_compile_targets": [
"telemetry_unittests_run"
],
"swarming": {
"can_use_on_swarming_builders": true
"can_use_on_swarming_builders": true,
"shards": 2
}
}
],
Expand Down
18 changes: 15 additions & 3 deletions testing/buildbot/chromium.linux.json
Original file line number Diff line number Diff line change
Expand Up @@ -3249,13 +3249,17 @@
}
},
{
"args": [
"--jobs=1"
],
"isolate_name": "telemetry_unittests",
"name": "telemetry_unittests",
"override_compile_targets": [
"telemetry_unittests_run"
],
"swarming": {
"can_use_on_swarming_builders": true
"can_use_on_swarming_builders": true,
"shards": 2
}
}
],
Expand Down Expand Up @@ -3804,13 +3808,17 @@
}
},
{
"args": [
"--jobs=1"
],
"isolate_name": "telemetry_unittests",
"name": "telemetry_unittests",
"override_compile_targets": [
"telemetry_unittests_run"
],
"swarming": {
"can_use_on_swarming_builders": true
"can_use_on_swarming_builders": true,
"shards": 2
}
}
],
Expand Down Expand Up @@ -4165,13 +4173,17 @@
}
},
{
"args": [
"--jobs=1"
],
"isolate_name": "telemetry_unittests",
"name": "telemetry_unittests",
"override_compile_targets": [
"telemetry_unittests_run"
],
"swarming": {
"can_use_on_swarming_builders": true
"can_use_on_swarming_builders": true,
"shards": 2
}
}
]
Expand Down

0 comments on commit 027b82c

Please sign in to comment.