Skip to content

Commit

Permalink
[lacros] Remove all experimental flags from lacros FYI builder
Browse files Browse the repository at this point in the history
This mainly gives me the ability to use resultdb to do some analysis
because experimental steps are not stored. crbug.com/1115614.

Bug: 1111979
Change-Id: Ie185dadc54d36ec6d2815652290806f477fb0298
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2505726
Reviewed-by: Yuke Liao <liaoyuke@chromium.org>
Commit-Queue: Sven Zheng <svenzheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#822021}
  • Loading branch information
Sven Zheng authored and Commit Bot committed Oct 29, 2020
1 parent 3a27264 commit a57118f
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 54 deletions.
11 changes: 0 additions & 11 deletions testing/buildbot/chromium.fyi.json
Original file line number Diff line number Diff line change
Expand Up @@ -58111,7 +58111,6 @@
"test_id_prefix": "ninja://third_party/boringssl:boringssl_ssl_tests/"
},
{
"experiment_percentage": 100,
"merge": {
"args": [],
"script": "//testing/merge_scripts/standard_gtest_merge.py"
Expand Down Expand Up @@ -58242,7 +58241,6 @@
"test_id_prefix": "ninja://components:components_browsertests/"
},
{
"experiment_percentage": 100,
"merge": {
"args": [],
"script": "//testing/merge_scripts/standard_gtest_merge.py"
Expand Down Expand Up @@ -58302,7 +58300,6 @@
"args": [
"--enable-features=StorageServiceOutOfProcess"
],
"experiment_percentage": 100,
"merge": {
"args": [],
"script": "//testing/merge_scripts/standard_gtest_merge.py"
Expand Down Expand Up @@ -58431,7 +58428,6 @@
"test_id_prefix": "ninja://ui/events:events_unittests/"
},
{
"experiment_percentage": 100,
"merge": {
"args": [],
"script": "//testing/merge_scripts/standard_gtest_merge.py"
Expand All @@ -58453,7 +58449,6 @@
"args": [
"--enable-features=StorageServiceOutOfProcess"
],
"experiment_percentage": 100,
"merge": {
"args": [],
"script": "//testing/merge_scripts/standard_gtest_merge.py"
Expand All @@ -58473,7 +58468,6 @@
"test_id_prefix": "ninja://extensions:extensions_browsertests/"
},
{
"experiment_percentage": 100,
"merge": {
"args": [],
"script": "//testing/merge_scripts/standard_gtest_merge.py"
Expand Down Expand Up @@ -58618,7 +58612,6 @@
"test_id_prefix": "ninja://components/gwp_asan:gwp_asan_unittests/"
},
{
"experiment_percentage": 100,
"merge": {
"args": [],
"script": "//testing/merge_scripts/standard_gtest_merge.py"
Expand All @@ -58641,7 +58634,6 @@
"args": [
"--enable-features=StorageServiceOutOfProcess"
],
"experiment_percentage": 100,
"merge": {
"args": [],
"script": "//testing/merge_scripts/standard_gtest_merge.py"
Expand Down Expand Up @@ -59184,7 +59176,6 @@
"test_id_prefix": "ninja://storage:storage_unittests/"
},
{
"experiment_percentage": 100,
"merge": {
"args": [],
"script": "//testing/merge_scripts/standard_gtest_merge.py"
Expand Down Expand Up @@ -59257,7 +59248,6 @@
"test_id_prefix": "ninja://ui/touch_selection:ui_touch_selection_unittests/"
},
{
"experiment_percentage": 100,
"merge": {
"args": [],
"script": "//testing/merge_scripts/standard_gtest_merge.py"
Expand Down Expand Up @@ -59294,7 +59284,6 @@
"test_id_prefix": "ninja://url:url_unittests/"
},
{
"experiment_percentage": 100,
"merge": {
"args": [],
"script": "//testing/merge_scripts/standard_gtest_merge.py"
Expand Down
43 changes: 0 additions & 43 deletions testing/buildbot/test_suite_exceptions.pyl
Original file line number Diff line number Diff line change
Expand Up @@ -636,9 +636,6 @@
'--test-launcher-filter-file=../../testing/buildbot/filters/lacros.browser_tests.filter',
],
},
'linux-lacros-tester-fyi-rel': {
'experiment_percentage': 100,
},
'linux-lacros-tester-rel': {
'args': [
'--test-launcher-filter-file=../../testing/buildbot/filters/lacros.browser_tests.filter',
Expand Down Expand Up @@ -965,10 +962,6 @@
],
},
# https://crbug.com/1111979,
'linux-lacros-tester-fyi-rel': {
'experiment_percentage': 100,
},
# https://crbug.com/1111979,
'linux-lacros-tester-rel': {
'args': [
'--gtest_filter=-NetExportFileWriterTest*',
Expand Down Expand Up @@ -1275,10 +1268,6 @@
'Win10 Tests x64 (dbg)': {
'experiment_percentage': 100, # https://crbug.com/876615
},
# https://crbug.com/1111979
'linux-lacros-tester-fyi-rel': {
'experiment_percentage': 100,
},
# https://crbug.com/1111979,
'linux-lacros-tester-rel': {
'args': [
Expand All @@ -1288,11 +1277,7 @@
},
},
'extensions_unittests': {
# https://crbug.com/1111979
'modifications': {
'linux-lacros-tester-fyi-rel': {
'experiment_percentage': 100,
},
# https://crbug.com/1111979,
'linux-lacros-tester-rel': {
'args': [
Expand Down Expand Up @@ -1591,10 +1576,6 @@
],
},
# https://crbug.com/1111979
'linux-lacros-tester-fyi-rel': {
'experiment_percentage': 100,
},
# https://crbug.com/1111979
'linux-lacros-tester-rel': {
'args': [
'--test-launcher-filter-file=../../testing/buildbot/filters/lacros.interactive_ui_tests.filter',
Expand Down Expand Up @@ -2521,10 +2502,6 @@
'shards': 6,
},
},
# https://crbug.com/1111979
'linux-lacros-tester-fyi-rel': {
'experiment_percentage': 100,
},
# https://crbug.com/1111979,
'linux-lacros-tester-rel': {
'experiment_percentage': 100,
Expand All @@ -2539,10 +2516,6 @@
},
'storage_service_extensions_browsertests': {
'modifications': {
# https://crbug.com/1111979
'linux-lacros-tester-fyi-rel': {
'experiment_percentage': 100,
},
# https://crbug.com/1111979,
'linux-lacros-tester-rel': {
'args': [
Expand All @@ -2562,10 +2535,6 @@
'--gtest_filter=-SadTabViewInteractiveUITest.ReloadMultipleSadTabs',
],
},
# https://crbug.com/1111979
'linux-lacros-tester-fyi-rel': {
'experiment_percentage': 100,
},
}
},
'swiftshader_unittests': {
Expand Down Expand Up @@ -2621,10 +2590,6 @@
'shards': 4,
},
},
# https://crbug.com/1111979
'linux-lacros-tester-fyi-rel': {
'experiment_percentage': 100,
},
# https://crbug.com/1111979,
'linux-lacros-tester-rel': {
'args': [
Expand Down Expand Up @@ -2823,10 +2788,6 @@
'--test-launcher-filter-file=../../testing/buildbot/filters/lacros.unit_tests.filter',
],
},
# https://crbug.com/1111979
'linux-lacros-tester-fyi-rel': {
'experiment_percentage': 100,
},
# https://crbug.com/1111979,
'linux-lacros-tester-rel': {
'args': [
Expand All @@ -2853,10 +2814,6 @@
},
'views_unittests': {
'modifications': {
# https://crbug.com/1111979
'linux-lacros-tester-fyi-rel': {
'experiment_percentage': 100,
},
# https://crbug.com/1111979,
'linux-lacros-tester-rel': {
'args': [
Expand Down

0 comments on commit a57118f

Please sign in to comment.