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

Unified Diff: recipe_modules/generator_script/api.py

Issue 2245113002: Track step nesting in StreamEngine. (Closed) Base URL: https://github.com/luci/recipes-py@emit-initial-properties
Patch Set: 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: recipe_modules/generator_script/api.py
diff --git a/recipe_modules/generator_script/api.py b/recipe_modules/generator_script/api.py
index ad9f634922e28805ce9690ac596a648555a29f7e..86d08066278069bea636bf66d3e5cf1617433c4a 100644
--- a/recipe_modules/generator_script/api.py
+++ b/recipe_modules/generator_script/api.py
@@ -30,8 +30,7 @@ class GeneratorScriptApi(recipe_api.RecipeApi):
infra_step: a bool which indicates that failures in this step are 'infra'
failures and should abort with a purple coloration instead red.
- step_nest_level: an integer which indicates that this step should appear
- on the build status page with this indentation level.
+ nested: true if this step is nested.
always_run: a bool which indicates that this step should run, even if
some previous step failed.
@@ -90,7 +89,6 @@ class GeneratorScriptApi(recipe_api.RecipeApi):
#TODO(martiniss) change this to use a regular step call
step['ok_ret'] = set(step.pop('ok_ret', {0}))
step['infra_step'] = bool(step.pop('infra_step', False))
- step['step_nest_level'] = int(step.pop('step_nest_level', 0))
dnj 2016/08/15 17:33:53 (This will be re-added in next PS, was oversight).
if step.pop('always_run', False) or not failed_steps:
try:

Powered by Google App Engine
This is Rietveld 408576698