Skip to content

Commit

Permalink
[infra] Remove Dart 1 builders from the CQ (Part I)
Browse files Browse the repository at this point in the history
* Remove obsolete vm-kernel-legacy and analyzer-cfe builders.

Change-Id: I6b316359b1a2c5ce2adcf276578a9b9be5db5c15
Reviewed-on: https://dart-review.googlesource.com/76480
Reviewed-by: William Hesse <whesse@google.com>
Commit-Queue: Alexander Thomas <athom@google.com>
  • Loading branch information
athomas authored and commit-bot@chromium.org committed Sep 25, 2018
1 parent 9185294 commit 8ff692a
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 54 deletions.
59 changes: 7 additions & 52 deletions tools/bots/test_matrix.json
Original file line number Diff line number Diff line change
Expand Up @@ -373,23 +373,20 @@
"options": {
"vm-options": ["--no-enable-malloc-hooks"]
}},
"dartk-(linux|mac)-(debug|release)-x64": { },
"dartk-(linux|mac)-(debug|release)-(ia32|x64)": { },
"dartk-checked-linux-release-x64": {
"options": {
"checked": true
}},
"dartk-win-release-x64": { },
"dartk-(linux|mac|win)-product-x64": { },
"dartk-linux-(debug|release)-simdbc64": { },
"dartk-linux-release-(simarm|simarm64)": { },
"dartk-optcounter-linux-release-x64": {
"options": {
"builder-tag": "optimization_counter_threshold",
"vm-options": ["--optimization-counter-threshold=5"]
}},
"dartk-legacy-linux-release-x64": {
"options": {
"preview-dart-2": false
}},
"dartk-reload-linux-(debug|release)-x64": {
"options": {
"hot-reload": true
Expand Down Expand Up @@ -432,12 +429,6 @@
"enable-asserts": true,
"use-sdk": true
}},
"analyzer-cfe-(linux|mac|win)": {
"options": {
"compiler": "dart2analyzer",
"use-cfe": true,
"use-sdk": true
}},
"analyzer-fasta_parser-linux": {
"options": {
"compiler": "dart2analyzer",
Expand Down Expand Up @@ -625,39 +616,17 @@
{
"name": "vm ia32 tests",
"arguments": [
"-nvm-legacy-linux-debug-ia32",
"-ndartk-linux-debug-ia32",
"vm"]
},
{
"name": "vm x64 tests",
"arguments": [
"-nvm-legacy-linux-debug-x64",
"-ndartk-linux-debug-x64",
"vm"]
}
]
},
{
"builders": [
"vm-kernel-legacy-linux-release-x64"
],
"meta": {
"description": "Tests the vm with dartk in legacy mode."
},
"steps": [
{
"name": "build dart",
"script": "tools/build.py",
"arguments": ["runtime_kernel"]
},
{
"name": "vm legacy tests",
"arguments": [
"-ndartk-legacy-linux-release-x64"],
"fileset": "vm-kernel",
"shards": 10
}
]
},
{
"builders": [
"vm-kernel-precomp-linux-debug-x64",
Expand Down Expand Up @@ -761,12 +730,15 @@
"builders": [
"vm-kernel-linux-debug-simdbc64",
"vm-kernel-linux-debug-x64",
"vm-kernel-linux-product-x64",
"vm-kernel-linux-release-simarm",
"vm-kernel-linux-release-simarm64",
"vm-kernel-linux-release-simdbc64",
"vm-kernel-linux-release-x64",
"vm-kernel-mac-debug-x64",
"vm-kernel-mac-product-x64",
"vm-kernel-mac-release-x64",
"vm-kernel-win-product-x64",
"vm-kernel-win-release-x64"
],
"meta": {
Expand Down Expand Up @@ -1621,23 +1593,6 @@
}
]
},
{
"builders": [
"analyzer-use-cfe-linux",
"analyzer-use-cfe-mac",
"analyzer-use-cfe-win"
],
"meta": {
"description": "The configuration used by the analyzer builders running --use-cfe."
},
"steps": [
{
"name": "build dart",
"script": "tools/build.py",
"arguments": ["create_sdk"]
}
]
},
{
"builders": ["analyzer-analysis-server-linux"],
"meta": {
Expand Down
2 changes: 0 additions & 2 deletions tools/infra/config/cq.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,6 @@ verifiers {
builders { name: "vm-kernel-linux-release-x64-try" }
builders { name: "vm-kernel-mac-release-x64-try" experiment_percentage: 5 }
builders { name: "vm-linux-product-x64-try" }
builders { name: "vm-linux-release-x64-try" }
builders { name: "vm-mac-release-x64-try" }
}
try_job_retry_config {
try_job_retry_quota: 0
Expand Down

0 comments on commit 8ff692a

Please sign in to comment.