Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(949)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: scripts/slave/recipes/bisect.expected/basic.json

Issue 2020543003: Revert of Avoid nesting steps called under waiting for x: (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « scripts/slave/recipes/bisect.py ('k') | scripts/slave/recipes/bisect.expected/basic_return_code_test.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/recipes/bisect.expected/basic.json
diff --git a/scripts/slave/recipes/bisect.expected/basic.json b/scripts/slave/recipes/bisect.expected/basic.json
index cec02fe7cfd6d1e728277623318314cd7349c083..976e20737bba037202e8dfc563936ae6fd671954 100644
--- a/scripts/slave/recipes/bisect.expected/basic.json
+++ b/scripts/slave/recipes/bisect.expected/basic.json
@@ -467,6 +467,27 @@
},
{
"cmd": [
+ "git",
+ "footers",
+ "--position",
+ "e28dc0d49c331def2a3bbf3ddd0096eb51551155"
+ ],
+ "cwd": "[SLAVE_BUILD]/src",
+ "name": "Gathering reference values.resolving hash e28dc0d49c331def2a3bbf3ddd0096eb51551155",
+ "stdout": "/path/to/tmp/",
+ "~followup_annotations": [
+ "@@@STEP_NEST_LEVEL@1@@@"
+ ]
+ },
+ {
+ "cmd": [],
+ "name": "Gathering reference values.Waiting for chromium@e28dc0d49c",
+ "~followup_annotations": [
+ "@@@STEP_NEST_LEVEL@1@@@"
+ ]
+ },
+ {
+ "cmd": [
"python",
"-u",
"RECIPE_MODULE[build::gsutil]/resources/gsutil_wrapper.py",
@@ -476,14 +497,14 @@
"ls",
"gs://chrome-perf/Linux Builder/full-build-linux_e28dc0d49c331def2a3bbf3ddd0096eb51551155.zip"
],
- "name": "Gathering reference values.gsutil ls (3)",
- "~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@"
- ]
- },
- {
- "cmd": [],
- "name": "Gathering reference values.Triggering test job for e28dc0d49c331def2a3bbf3ddd0096eb51551155",
+ "name": "Gathering reference values.Waiting for chromium@e28dc0d49c.gsutil ls",
+ "~followup_annotations": [
+ "@@@STEP_NEST_LEVEL@2@@@"
+ ]
+ },
+ {
+ "cmd": [],
+ "name": "Gathering reference values.Waiting for chromium@e28dc0d49c.Triggering test job for e28dc0d49c331def2a3bbf3ddd0096eb51551155",
"trigger_specs": [
{
"builder_name": "linux_perf_tester",
@@ -505,7 +526,7 @@
}
],
"~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@"
+ "@@@STEP_NEST_LEVEL@2@@@"
]
},
{
@@ -514,9 +535,9 @@
"-u",
"\nimport sys\nimport time\ntime.sleep(20*60)\nsys.exit(0)\n"
],
- "name": "Gathering reference values.sleeping",
- "~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@",
+ "name": "Gathering reference values.Waiting for chromium@e28dc0d49c.sleeping",
+ "~followup_annotations": [
+ "@@@STEP_NEST_LEVEL@2@@@",
"@@@STEP_LOG_LINE@python.inline@@@@",
"@@@STEP_LOG_LINE@python.inline@import sys@@@",
"@@@STEP_LOG_LINE@python.inline@import time@@@",
@@ -536,9 +557,9 @@
"ls",
"gs://chrome-perf/bisect-results/e28dc0d49c331def2a3bbf3ddd0096eb51551155-test.results"
],
- "name": "Gathering reference values.gsutil ls (4)",
- "~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@"
+ "name": "Gathering reference values.Waiting for chromium@e28dc0d49c.gsutil ls (2)",
+ "~followup_annotations": [
+ "@@@STEP_NEST_LEVEL@2@@@"
]
},
{
@@ -552,8 +573,29 @@
"cat",
"gs://chrome-perf/bisect-results/e28dc0d49c331def2a3bbf3ddd0096eb51551155-test.results"
],
- "name": "Gathering reference values.gsutil Get test results for build e28dc0d49c331def2a3bbf3ddd0096eb51551155",
+ "name": "Gathering reference values.Waiting for chromium@e28dc0d49c.gsutil Get test results for build e28dc0d49c331def2a3bbf3ddd0096eb51551155",
"stdout": "/path/to/tmp/",
+ "~followup_annotations": [
+ "@@@STEP_NEST_LEVEL@2@@@"
+ ]
+ },
+ {
+ "cmd": [
+ "git",
+ "footers",
+ "--position",
+ "a6298e4afedbf2cd461755ea6f45b0ad64222222"
+ ],
+ "cwd": "[SLAVE_BUILD]/src",
+ "name": "Gathering reference values.resolving hash a6298e4afedbf2cd461755ea6f45b0ad64222222",
+ "stdout": "/path/to/tmp/",
+ "~followup_annotations": [
+ "@@@STEP_NEST_LEVEL@1@@@"
+ ]
+ },
+ {
+ "cmd": [],
+ "name": "Gathering reference values.Waiting for chromium@a6298e4afe",
"~followup_annotations": [
"@@@STEP_NEST_LEVEL@1@@@"
]
@@ -569,14 +611,14 @@
"ls",
"gs://chrome-perf/Linux Builder/full-build-linux_a6298e4afedbf2cd461755ea6f45b0ad64222222.zip"
],
- "name": "Gathering reference values.gsutil ls (5)",
- "~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@"
- ]
- },
- {
- "cmd": [],
- "name": "Gathering reference values.Triggering test job for a6298e4afedbf2cd461755ea6f45b0ad64222222",
+ "name": "Gathering reference values.Waiting for chromium@a6298e4afe.gsutil ls",
+ "~followup_annotations": [
+ "@@@STEP_NEST_LEVEL@2@@@"
+ ]
+ },
+ {
+ "cmd": [],
+ "name": "Gathering reference values.Waiting for chromium@a6298e4afe.Triggering test job for a6298e4afedbf2cd461755ea6f45b0ad64222222",
"trigger_specs": [
{
"builder_name": "linux_perf_tester",
@@ -598,7 +640,7 @@
}
],
"~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@"
+ "@@@STEP_NEST_LEVEL@2@@@"
]
},
{
@@ -607,9 +649,9 @@
"-u",
"\nimport sys\nimport time\ntime.sleep(20*60)\nsys.exit(0)\n"
],
- "name": "Gathering reference values.sleeping (2)",
- "~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@",
+ "name": "Gathering reference values.Waiting for chromium@a6298e4afe.sleeping",
+ "~followup_annotations": [
+ "@@@STEP_NEST_LEVEL@2@@@",
"@@@STEP_LOG_LINE@python.inline@@@@",
"@@@STEP_LOG_LINE@python.inline@import sys@@@",
"@@@STEP_LOG_LINE@python.inline@import time@@@",
@@ -629,9 +671,9 @@
"ls",
"gs://chrome-perf/bisect-results/a6298e4afedbf2cd461755ea6f45b0ad64222222-test.results"
],
- "name": "Gathering reference values.gsutil ls (6)",
- "~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@"
+ "name": "Gathering reference values.Waiting for chromium@a6298e4afe.gsutil ls (2)",
+ "~followup_annotations": [
+ "@@@STEP_NEST_LEVEL@2@@@"
]
},
{
@@ -645,10 +687,10 @@
"cat",
"gs://chrome-perf/bisect-results/a6298e4afedbf2cd461755ea6f45b0ad64222222-test.results"
],
- "name": "Gathering reference values.gsutil Get test results for build a6298e4afedbf2cd461755ea6f45b0ad64222222",
+ "name": "Gathering reference values.Waiting for chromium@a6298e4afe.gsutil Get test results for build a6298e4afedbf2cd461755ea6f45b0ad64222222",
"stdout": "/path/to/tmp/",
"~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@"
+ "@@@STEP_NEST_LEVEL@2@@@"
]
},
{
@@ -669,34 +711,6 @@
{
"cmd": [],
"name": "Working on revision chromium@00316c9ddf"
- },
- {
- "cmd": [
- "git",
- "footers",
- "--position",
- "e28dc0d49c331def2a3bbf3ddd0096eb51551155"
- ],
- "cwd": "[SLAVE_BUILD]/src",
- "name": "Working on revision chromium@00316c9ddf.resolving hash e28dc0d49c331def2a3bbf3ddd0096eb51551155",
- "stdout": "/path/to/tmp/",
- "~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@"
- ]
- },
- {
- "cmd": [
- "git",
- "footers",
- "--position",
- "a6298e4afedbf2cd461755ea6f45b0ad64222222"
- ],
- "cwd": "[SLAVE_BUILD]/src",
- "name": "Working on revision chromium@00316c9ddf.resolving hash a6298e4afedbf2cd461755ea6f45b0ad64222222",
- "stdout": "/path/to/tmp/",
- "~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@"
- ]
},
{
"cmd": [
@@ -781,6 +795,13 @@
]
},
{
+ "cmd": [],
+ "name": "Working on revision chromium@00316c9ddf.Waiting for chromium@00316c9ddf",
+ "~followup_annotations": [
+ "@@@STEP_NEST_LEVEL@1@@@"
+ ]
+ },
+ {
"cmd": [
"python",
"-u",
@@ -791,14 +812,14 @@
"ls",
"gs://chrome-perf/Linux Builder/full-build-linux_00316c9ddfb9d7b4e1ed2fff9fe6d964d2111111.zip"
],
- "name": "Working on revision chromium@00316c9ddf.gsutil ls (2)",
- "~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@"
- ]
- },
- {
- "cmd": [],
- "name": "Working on revision chromium@00316c9ddf.Triggering test job for 00316c9ddfb9d7b4e1ed2fff9fe6d964d2111111",
+ "name": "Working on revision chromium@00316c9ddf.Waiting for chromium@00316c9ddf.gsutil ls",
+ "~followup_annotations": [
+ "@@@STEP_NEST_LEVEL@2@@@"
+ ]
+ },
+ {
+ "cmd": [],
+ "name": "Working on revision chromium@00316c9ddf.Waiting for chromium@00316c9ddf.Triggering test job for 00316c9ddfb9d7b4e1ed2fff9fe6d964d2111111",
"trigger_specs": [
{
"builder_name": "linux_perf_tester",
@@ -820,7 +841,7 @@
}
],
"~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@"
+ "@@@STEP_NEST_LEVEL@2@@@"
]
},
{
@@ -829,9 +850,9 @@
"-u",
"\nimport sys\nimport time\ntime.sleep(20*60)\nsys.exit(0)\n"
],
- "name": "Working on revision chromium@00316c9ddf.sleeping",
- "~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@",
+ "name": "Working on revision chromium@00316c9ddf.Waiting for chromium@00316c9ddf.sleeping",
+ "~followup_annotations": [
+ "@@@STEP_NEST_LEVEL@2@@@",
"@@@STEP_LOG_LINE@python.inline@@@@",
"@@@STEP_LOG_LINE@python.inline@import sys@@@",
"@@@STEP_LOG_LINE@python.inline@import time@@@",
@@ -851,9 +872,9 @@
"ls",
"gs://chrome-perf/bisect-results/00316c9ddfb9d7b4e1ed2fff9fe6d964d2111111-test.results"
],
- "name": "Working on revision chromium@00316c9ddf.gsutil ls (3)",
- "~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@"
+ "name": "Working on revision chromium@00316c9ddf.Waiting for chromium@00316c9ddf.gsutil ls (2)",
+ "~followup_annotations": [
+ "@@@STEP_NEST_LEVEL@2@@@"
]
},
{
@@ -867,10 +888,10 @@
"cat",
"gs://chrome-perf/bisect-results/00316c9ddfb9d7b4e1ed2fff9fe6d964d2111111-test.results"
],
- "name": "Working on revision chromium@00316c9ddf.gsutil Get test results for build 00316c9ddfb9d7b4e1ed2fff9fe6d964d2111111",
+ "name": "Working on revision chromium@00316c9ddf.Waiting for chromium@00316c9ddf.gsutil Get test results for build 00316c9ddfb9d7b4e1ed2fff9fe6d964d2111111",
"stdout": "/path/to/tmp/",
"~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@"
+ "@@@STEP_NEST_LEVEL@2@@@"
]
},
{
@@ -882,10 +903,10 @@
"[14, 15, 16]",
"0.01"
],
- "name": "Working on revision chromium@00316c9ddf.Checking sample difference",
+ "name": "Working on revision chromium@00316c9ddf.Waiting for chromium@00316c9ddf.Checking sample difference",
"stdout": "/path/to/tmp/json",
"~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@",
+ "@@@STEP_NEST_LEVEL@2@@@",
"@@@STEP_LOG_LINE@json.output (invalid)@null@@@",
"@@@STEP_LOG_END@json.output (invalid)@@@"
]
@@ -899,10 +920,10 @@
"[14, 15, 16]",
"0.01"
],
- "name": "Working on revision chromium@00316c9ddf.Checking sample difference (2)",
+ "name": "Working on revision chromium@00316c9ddf.Waiting for chromium@00316c9ddf.Checking sample difference (2)",
"stdout": "/path/to/tmp/json",
"~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@",
+ "@@@STEP_NEST_LEVEL@2@@@",
"@@@STEP_LOG_LINE@json.output (invalid)@null@@@",
"@@@STEP_LOG_END@json.output (invalid)@@@"
]
@@ -1005,6 +1026,13 @@
]
},
{
+ "cmd": [],
+ "name": "Working on revision chromium@fc6dfc7ff5.Waiting for chromium@fc6dfc7ff5",
+ "~followup_annotations": [
+ "@@@STEP_NEST_LEVEL@1@@@"
+ ]
+ },
+ {
"cmd": [
"python",
"-u",
@@ -1015,14 +1043,14 @@
"ls",
"gs://chrome-perf/Linux Builder/full-build-linux_fc6dfc7ff5b1073408499478969261b826441144.zip"
],
- "name": "Working on revision chromium@fc6dfc7ff5.gsutil ls (2)",
- "~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@"
- ]
- },
- {
- "cmd": [],
- "name": "Working on revision chromium@fc6dfc7ff5.Triggering test job for fc6dfc7ff5b1073408499478969261b826441144",
+ "name": "Working on revision chromium@fc6dfc7ff5.Waiting for chromium@fc6dfc7ff5.gsutil ls",
+ "~followup_annotations": [
+ "@@@STEP_NEST_LEVEL@2@@@"
+ ]
+ },
+ {
+ "cmd": [],
+ "name": "Working on revision chromium@fc6dfc7ff5.Waiting for chromium@fc6dfc7ff5.Triggering test job for fc6dfc7ff5b1073408499478969261b826441144",
"trigger_specs": [
{
"builder_name": "linux_perf_tester",
@@ -1044,7 +1072,7 @@
}
],
"~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@"
+ "@@@STEP_NEST_LEVEL@2@@@"
]
},
{
@@ -1053,9 +1081,9 @@
"-u",
"\nimport sys\nimport time\ntime.sleep(20*60)\nsys.exit(0)\n"
],
- "name": "Working on revision chromium@fc6dfc7ff5.sleeping",
- "~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@",
+ "name": "Working on revision chromium@fc6dfc7ff5.Waiting for chromium@fc6dfc7ff5.sleeping",
+ "~followup_annotations": [
+ "@@@STEP_NEST_LEVEL@2@@@",
"@@@STEP_LOG_LINE@python.inline@@@@",
"@@@STEP_LOG_LINE@python.inline@import sys@@@",
"@@@STEP_LOG_LINE@python.inline@import time@@@",
@@ -1075,9 +1103,9 @@
"ls",
"gs://chrome-perf/bisect-results/fc6dfc7ff5b1073408499478969261b826441144-test.results"
],
- "name": "Working on revision chromium@fc6dfc7ff5.gsutil ls (3)",
- "~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@"
+ "name": "Working on revision chromium@fc6dfc7ff5.Waiting for chromium@fc6dfc7ff5.gsutil ls (2)",
+ "~followup_annotations": [
+ "@@@STEP_NEST_LEVEL@2@@@"
]
},
{
@@ -1091,10 +1119,10 @@
"cat",
"gs://chrome-perf/bisect-results/fc6dfc7ff5b1073408499478969261b826441144-test.results"
],
- "name": "Working on revision chromium@fc6dfc7ff5.gsutil Get test results for build fc6dfc7ff5b1073408499478969261b826441144",
+ "name": "Working on revision chromium@fc6dfc7ff5.Waiting for chromium@fc6dfc7ff5.gsutil Get test results for build fc6dfc7ff5b1073408499478969261b826441144",
"stdout": "/path/to/tmp/",
"~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@"
+ "@@@STEP_NEST_LEVEL@2@@@"
]
},
{
@@ -1106,10 +1134,10 @@
"[68, 70, 72]",
"0.01"
],
- "name": "Working on revision chromium@fc6dfc7ff5.Checking sample difference",
+ "name": "Working on revision chromium@fc6dfc7ff5.Waiting for chromium@fc6dfc7ff5.Checking sample difference",
"stdout": "/path/to/tmp/json",
"~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@",
+ "@@@STEP_NEST_LEVEL@2@@@",
"@@@STEP_LOG_LINE@json.output (invalid)@null@@@",
"@@@STEP_LOG_END@json.output (invalid)@@@"
]
@@ -1123,10 +1151,10 @@
"[68, 70, 72]",
"0.01"
],
- "name": "Working on revision chromium@fc6dfc7ff5.Checking sample difference (2)",
+ "name": "Working on revision chromium@fc6dfc7ff5.Waiting for chromium@fc6dfc7ff5.Checking sample difference (2)",
"stdout": "/path/to/tmp/json",
"~followup_annotations": [
- "@@@STEP_NEST_LEVEL@1@@@",
+ "@@@STEP_NEST_LEVEL@2@@@",
"@@@STEP_LOG_LINE@json.output (invalid)@null@@@",
"@@@STEP_LOG_END@json.output (invalid)@@@"
]
« no previous file with comments | « scripts/slave/recipes/bisect.py ('k') | scripts/slave/recipes/bisect.expected/basic_return_code_test.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698