New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 822141 link

Starred by 1 user
Status: Verified
Owner:
lijeffrey@chromium.org
Closed: Apr 2018
Cc:
█ robert...@chromium.org
chanli@chromium.org
lijeffrey@chromium.org
liaoyuke@chromium.org
st...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

[Findit] Flake Analyzer - GetIsolateShaForCommitPositionPipeline returns entire try job result

Project Member Reported by lijeffrey@chromium.org, Mar 15 2018 
Only the isolate sha should be yielded, not the entire report
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 15 2018 

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/7e85ec76547de7dd7d62659ae4a944b52644eb74

commit 7e85ec76547de7dd7d62659ae4a944b52644eb74
Author: Jeffrey Li <lijeffrey@chromium.org>
Date: Thu Mar 15 23:06:41 2018

[Findit] Flake Analyzer - Return isolate sha from flake try job result

GetIsolateShaForCommitPositionPipeline is returning the entire try job
result instead of just the isolate sha. Adding a synchronous pipeline
to extract the sha from the try job result and return it properly.

Bug:  822141 
Change-Id: I3509e74dfa81825d189a71b9ee048900955c2dc8
Reviewed-on: https://chromium-review.googlesource.com/963769
Commit-Queue: Jeffrey Li <lijeffrey@chromium.org>
Reviewed-by: Brandon Wylie <wylieb@chromium.org>

[modify] https://crrev.com/7e85ec76547de7dd7d62659ae4a944b52644eb74/appengine/findit/pipelines/flake_failure/test/get_isolate_sha_pipeline_test.py
[modify] https://crrev.com/7e85ec76547de7dd7d62659ae4a944b52644eb74/appengine/findit/pipelines/flake_failure/get_isolate_sha_pipeline.py
[modify] https://crrev.com/7e85ec76547de7dd7d62659ae4a944b52644eb74/appengine/findit/dto/isolated_tests.py

Comment 2 by lijeffrey@chromium.org, Apr 4 2018

Status: Verified (was: Assigned)
►

Sign in to add a comment