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

Unified Diff: scripts/slave/recipe_modules/auto_bisect/example.py

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
Index: scripts/slave/recipe_modules/auto_bisect/example.py
diff --git a/scripts/slave/recipe_modules/auto_bisect/example.py b/scripts/slave/recipe_modules/auto_bisect/example.py
index 7727649afad1fa84439259d2d3faa6b6e8ddbdcb..e96b9a0ac21f25f5f2bf75b161dfbbfdc1ecc5f3 100644
--- a/scripts/slave/recipe_modules/auto_bisect/example.py
+++ b/scripts/slave/recipe_modules/auto_bisect/example.py
@@ -87,13 +87,13 @@
api, _get_ref_range_only_test_data(), 'failed_build_test',
extra_config={'dummy_builds': None})
failed_build_test += api.step_data('gsutil ls', retcode=1)
- failed_build_test += api.step_data('gsutil ls (2)' , retcode=1)
- failed_build_test += api.step_data('gsutil ls (3)' , retcode=1)
failed_build_test += api.step_data(
- 'fetch builder state',
+ 'Waiting for chromium@a6298e4afe.gsutil ls' , retcode=1)
+ failed_build_test += api.step_data(
+ 'Waiting for chromium@a6298e4afe.fetch builder state',
api.raw_io.output(json.dumps({'cachedBuilds': ['2106']})))
failed_build_test += api.step_data(
- 'fetch build details',
+ 'Waiting for chromium@a6298e4afe.fetch build details',
api.raw_io.output(json.dumps({
'results': 2,
'properties': [('build_archive_url',
@@ -107,26 +107,27 @@
api, _get_ref_range_only_test_data(), 'delayed_build_test',
extra_config={'dummy_builds': None})
delayed_build_test += api.step_data('gsutil ls', retcode=1)
- delayed_build_test += api.step_data('gsutil ls (2)', retcode=1)
- delayed_build_test += api.step_data('gsutil ls (3)', retcode=1)
- delayed_build_test += api.step_data('gsutil ls (4)', retcode=1)
- delayed_build_test += api.step_data('gsutil ls (5)', retcode=1)
- delayed_build_test += api.step_data('gsutil ls (6)', retcode=1)
- delayed_build_test += api.step_data(
- 'fetch builder state',
+ delayed_build_test += api.step_data(
+ 'Waiting for chromium@a6298e4afe.gsutil ls', retcode=1)
+ delayed_build_test += api.step_data(
+ 'Waiting for chromium@a6298e4afe.gsutil ls (2)', retcode=1)
+ delayed_build_test += api.step_data(
+ 'Waiting for chromium@a6298e4afe.gsutil ls (3)', retcode=1)
+ delayed_build_test += api.step_data(
+ 'Waiting for chromium@a6298e4afe.fetch builder state',
api.raw_io.output(json.dumps({'cachedBuilds': []})))
delayed_build_test += api.step_data(
- 'fetch builder state (2)',
+ 'Waiting for chromium@a6298e4afe.fetch builder state (2)',
api.raw_io.output(json.dumps({'cachedBuilds': ['2106']})))
delayed_build_test += api.step_data(
- 'fetch build details',
+ 'Waiting for chromium@a6298e4afe.fetch build details',
api.raw_io.output(json.dumps({
'properties': [('build_archive_url',
('gs://chrome-perf/Linux Builder/full-build-linux_'
'a6298e4afedbf2cd461755ea6f45b0ad64222222.zip'))]
})))
delayed_build_test += api.step_data(
- 'fetch build details (2)',
+ 'Waiting for chromium@a6298e4afe.fetch build details (2)',
api.raw_io.output(json.dumps({
'results': 2,
'properties': [('build_archive_url',
@@ -429,7 +430,9 @@
if include_build_steps:
if test_results:
- step_name = ('gsutil Get test results for build %s') % commit_hash
+ step_name = ('Waiting for chromium@%s.gsutil '
+ 'Get test results for build %s') % (commit_hash[:10],
+ commit_hash)
yield api.step_data(step_name, stdout=api.json.output(test_results))
if revision_data.get('DEPS', False):

Powered by Google App Engine
This is Rietveld 408576698