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

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

Issue 2237403002: Add cloudtail start/stop in recipe_modules/goma (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: remove import os Created 4 years, 4 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/goma/example.py
diff --git a/scripts/slave/recipe_modules/goma/example.py b/scripts/slave/recipe_modules/goma/example.py
index 09eed4db491426fbbbe08c3cd20c32345eb5559a..e09946f3492343a2628b2de4832eec9a797b36b5 100644
--- a/scripts/slave/recipe_modules/goma/example.py
+++ b/scripts/slave/recipe_modules/goma/example.py
@@ -6,6 +6,7 @@ DEPS = [
'goma',
'recipe_engine/platform',
'recipe_engine/properties',
+ 'recipe_engine/raw_io',
]
def RunSteps(api):
@@ -22,7 +23,11 @@ def RunSteps(api):
def GenTests(api):
for platform in ('linux', 'win', 'mac'):
- yield api.test(platform) + api.platform.name(platform)
+ step_data = api.step_data(
+ 'start cloudtail',
+ stdout=api.raw_io.output('12345')
+ )
+ yield api.test(platform) + api.platform.name(platform) + step_data
properties = {
'build_data_dir': 'build_data_dir',
@@ -33,4 +38,4 @@ def GenTests(api):
}
yield (api.test('%s_upload_logs' % platform) + api.platform.name(platform) +
- api.properties.generic(**properties))
+ api.properties.generic(**properties) + step_data)

Powered by Google App Engine
This is Rietveld 408576698