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

Unified Diff: scripts/slave/recipe_modules/bisect_tester_staging/perf_test.py

Issue 2456483005: Treat bisect/tryjob perf test failures as warnings. (Closed)
Patch Set: Created 4 years, 2 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/bisect_tester_staging/perf_test.py
diff --git a/scripts/slave/recipe_modules/bisect_tester_staging/perf_test.py b/scripts/slave/recipe_modules/bisect_tester_staging/perf_test.py
index 8c61ebabcb577b35a38f094bd3718e50afe92dd4..9c9890c64459f9b144f9ed3102c834b20e0c1f97 100644
--- a/scripts/slave/recipe_modules/bisect_tester_staging/perf_test.py
+++ b/scripts/slave/recipe_modules/bisect_tester_staging/perf_test.py
@@ -204,6 +204,7 @@ def _run_command(api, command, step_name, **kwargs):
step_result.presentation.logs['Captured Output'] = (
step_result.stdout or '').splitlines()
except api.m.step.StepFailure as sf:
+ sf.result.presentation.status = api.m.step.WARNING
sf.result.presentation.logs['Failure Output'] = (
sf.result.stdout or '').splitlines()
if sf.result.stderr: # pragma: no cover

Powered by Google App Engine
This is Rietveld 408576698