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

Unified Diff: recipe_engine/run.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
« no previous file with comments | « no previous file | recipe_engine/simulation_test.py » ('j') | recipe_engine/simulation_test.py » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: recipe_engine/run.py
diff --git a/recipe_engine/run.py b/recipe_engine/run.py
index 17b8890053c65d2ba19bda1dc8ce5adac8914e05..385050d05148c2d3246767e64c15700e023db69a 100644
--- a/recipe_engine/run.py
+++ b/recipe_engine/run.py
@@ -296,8 +296,8 @@ class RecipeEngine(object):
* step - uses engine.create_step(...), and previous_step_result.
"""
- ActiveStep = collections.namedtuple('ActiveStep',
- 'step step_result open_step nest_level')
+ ActiveStep = collections.namedtuple('ActiveStep', (
+ 'step', 'step_result', 'open_step', 'nest_level'))
def __init__(self, step_runner, properties, universe_view):
"""See run_steps() for parameter meanings."""
@@ -368,7 +368,7 @@ class RecipeEngine(object):
step_result = None
- nest_level = step.pop('step_nest_level', 0)
+ nest_level = step.get('step_nest_level', 0)
self._close_to_level(nest_level)
open_step = self._step_runner.open_step(step)
@@ -377,8 +377,6 @@ class RecipeEngine(object):
step_result=None,
open_step=open_step,
nest_level=nest_level))
- if nest_level:
- open_step.stream.set_nest_level(nest_level)
step_result = open_step.run()
self._step_stack[-1] = (
« no previous file with comments | « no previous file | recipe_engine/simulation_test.py » ('j') | recipe_engine/simulation_test.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698