Commit d3af6994 authored by Debrian Figueroa's avatar Debrian Figueroa Committed by Commit Bot

Add failure status on timeouts

Change-Id: I6f7b0791172f9f4e7af278f8acb3f1ca986789d3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1700390
Commit-Queue: Debrian Figueroa <debrian@google.com>
Reviewed-by: 's avatarStephen Martinis <martiniss@chromium.org>
Reviewed-by: 's avatarRobbie Iannucci <iannucci@chromium.org>
parent 10e07dd3
......@@ -129,7 +129,7 @@ class PresubmitApi(recipe_api.RecipeApi):
self.m.step.active_result.presentation.status = 'FAILURE'
if self.m.step.active_result.exc_result.had_timeout:
# TODO(iannucci): Shouldn't we also mark failure on timeouts?
raw_result.status = common_pb2.FAILURE
raw_result.summary_markdown += (
'\n\nTimeout occurred during presubmit step.')
elif retcode == 1:
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment