Commit ff615348 authored by Lei Zhang's avatar Lei Zhang Committed by Commit Bot

Revert "Add builder for opus test in prevision of rolls."

This reverts commit 9db0a32a.

Reason for revert: Breaking Deterministic {Linux,Windows} bots.

Original change's description:
> Add builder for opus test in prevision of rolls.
> 
> This CL introduces an ad-hoc builder for roll of opus into chromium.
>   1/ Used as pres-submit validation.
>   2/ Runs dedicated tests and benchmarks for a fixed Android model.
> 
> The point 2/ shall be completed in follow-up CLs.
> 
> Bug: chromium:1002973
> Change-Id: Idb5c8b46ab68d8a9d63a864594396ac021a22746
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1816555
> Commit-Queue: Yves Gerey <yvesg@google.com>
> Reviewed-by: John Budorick <jbudorick@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#704847}

TBR=agable@chromium.org,jbudorick@chromium.org,yvesg@google.com

Change-Id: Ie6631651d93f83780bbf81b8291f3a9e3fd2bab5
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: chromium:1002973
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1854686Reviewed-by: default avatarLei Zhang <thestig@chromium.org>
Reviewed-by: default avatarJohn Budorick <jbudorick@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#705107}
parent dfdf6d7b
...@@ -3953,12 +3953,6 @@ buckets { ...@@ -3953,12 +3953,6 @@ buckets {
} }
# Keep builders sorted by OS, then name. # Keep builders sorted by OS, then name.
builders {
mixins: "android-try"
mixins: "builderless"
mixins: "linux-xenial"
name: "android-opus-kitkat-arm-rel"
}
builders { builders {
mixins: "android-angle-try" mixins: "android-angle-try"
mixins: "goma-rbe-prod" mixins: "goma-rbe-prod"
......
...@@ -3953,12 +3953,6 @@ buckets { ...@@ -3953,12 +3953,6 @@ buckets {
} }
# Keep builders sorted by OS, then name. # Keep builders sorted by OS, then name.
builders {
mixins: "android-try"
mixins: "builderless"
mixins: "linux-xenial"
name: "android-opus-kitkat-arm-rel"
}
builders { builders {
mixins: "android-angle-try" mixins: "android-angle-try"
mixins: "goma-rbe-prod" mixins: "goma-rbe-prod"
......
...@@ -4132,9 +4132,6 @@ consoles { ...@@ -4132,9 +4132,6 @@ consoles {
repo_url: "https://chromium.googlesource.com/chromium/src" repo_url: "https://chromium.googlesource.com/chromium/src"
refs: "refs/heads/master" refs: "refs/heads/master"
manifest_name: "REVISION" manifest_name: "REVISION"
builders {
name: "buildbucket/luci.chromium.try/android-opus-kitkat-arm-rel"
}
builders { builders {
name: "buildbucket/luci.chromium.try/android_archive_rel_ng" name: "buildbucket/luci.chromium.try/android_archive_rel_ng"
} }
......
...@@ -4132,9 +4132,6 @@ consoles { ...@@ -4132,9 +4132,6 @@ consoles {
repo_url: "https://chromium.googlesource.com/chromium/src" repo_url: "https://chromium.googlesource.com/chromium/src"
refs: "refs/heads/master" refs: "refs/heads/master"
manifest_name: "REVISION" manifest_name: "REVISION"
builders {
name: "buildbucket/luci.chromium.try/android-opus-kitkat-arm-rel"
}
builders { builders {
name: "buildbucket/luci.chromium.try/android_archive_rel_ng" name: "buildbucket/luci.chromium.try/android_archive_rel_ng"
} }
......
...@@ -2103,11 +2103,6 @@ ...@@ -2103,11 +2103,6 @@
"label": "//ios/testing:ocmock_support_unittests", "label": "//ios/testing:ocmock_support_unittests",
"type": "raw", "type": "raw",
}, },
"opus_decoder_unittests": {
"args": [],
"label": "//third_party/opus:test_opus_decode",
"type": "raw",
},
"origin_policy_parser_fuzzer": { "origin_policy_parser_fuzzer": {
"label": "//content/test/fuzzer:origin_policy_parser_fuzzer", "label": "//content/test/fuzzer:origin_policy_parser_fuzzer",
"type": "fuzzer", "type": "fuzzer",
......
...@@ -4570,13 +4570,6 @@ ...@@ -4570,13 +4570,6 @@
} }
}, },
'opus_roll_validation_tests': {
'opus_decoder_unittests': {
# Ad-hoc binary which returns 0 on success.
'test': 'audio_decoder_unittests',
},
},
'ozone_linux_gtests': { 'ozone_linux_gtests': {
'services_unittests': {}, 'services_unittests': {},
'ozone_unittests': {}, 'ozone_unittests': {},
......
...@@ -376,30 +376,6 @@ ...@@ -376,30 +376,6 @@
} }
] ]
}, },
"android-opus-kitkat-arm-rel": {
"isolated_scripts": [
{
"isolate_name": "opus_decoder_unittests",
"merge": {
"args": [],
"script": "//testing/merge_scripts/standard_isolated_script_merge.py"
},
"name": "opus_decoder_unittests",
"swarming": {
"can_use_on_swarming_builders": true,
"dimension_sets": [
{
"device_os": "KTU84P",
"device_os_type": "userdebug",
"device_type": "hammerhead",
"os": "Android"
}
]
},
"test": "audio_decoder_unittests"
}
]
},
"android_blink_rel": { "android_blink_rel": {
"gtest_tests": [ "gtest_tests": [
{ {
......
...@@ -4430,16 +4430,6 @@ ...@@ -4430,16 +4430,6 @@
}, },
'os_type': 'android', 'os_type': 'android',
}, },
'android-opus-kitkat-arm-rel': {
'mixins': [
'kitkat',
'hammerhead',
],
'os_type': 'android',
'test_suites': {
'isolated_scripts': 'opus_roll_validation_tests',
},
},
'android_blink_rel': { 'android_blink_rel': {
'mixins': [ 'mixins': [
'kitkat', 'kitkat',
......
...@@ -650,7 +650,6 @@ ...@@ -650,7 +650,6 @@
}, },
'tryserver.chromium.android': { 'tryserver.chromium.android': {
'android-opus-kitkat-arm-rel': 'android_release_trybot',
# TODO(crbug/597596): Switch this back to debug_trybot when cronet's # TODO(crbug/597596): Switch this back to debug_trybot when cronet's
# shared library loading is fixed. # shared library loading is fixed.
'android-cronet-arm-dbg': 'android_cronet_debug_static_bot_arm_no_neon', 'android-cronet-arm-dbg': 'android_cronet_debug_static_bot_arm_no_neon',
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment