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

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

Issue 1801843002: Revert of Use the crrev recipe module to fetch commit hashes. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 4 years, 9 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 c66bac66c2eeb82f4175bbdd94d4716a31103156..889dbf5bf2f58b0740195474ebaeabacfbbdf8aa 100644
--- a/scripts/slave/recipe_modules/auto_bisect/example.py
+++ b/scripts/slave/recipe_modules/auto_bisect/example.py
@@ -359,17 +359,16 @@
def _get_step_data_for_revision(api, revision_data, include_build_steps=True):
"""Generator that produces step patches for fake results."""
- commit_pos_number = revision_data['commit_pos']
+ commit_pos = revision_data['commit_pos']
commit_hash = revision_data['hash']
test_results = revision_data.get('test_results')
if 'refrange' in revision_data:
parent_step = 'Resolving reference range.'
- commit_pos = 'refs/heads/master@{#%s}' % commit_pos_number
- step_name = parent_step + 'crrev get commit hash for ' + commit_pos
- yield api.step_data(
- step_name,
- stdout=api.json.output({'git_sha': commit_hash}))
+ step_name = parent_step + 'resolving commit_pos ' + commit_pos
+ yield api.step_data(step_name,
+ stdout=api.raw_io.output('hash:' + commit_hash))
+
if include_build_steps:
if test_results:

Powered by Google App Engine
This is Rietveld 408576698