Culprit on SoM was not displayed correctly |
||||
Issue descriptionIn this build https://findit-for-me.appspot.com/waterfall/build-failure?url=https%3A%2F%2Fbuild.chromium.org%2Fp%2Fchromium.mac%2Fbuilders%2FMac10.11%2520Tests%2Fbuilds%2F2871 , the cluprits found by Findit were only for step "unit_tests", but on SoM they were shown for step "browser_tests" as well https://screenshot.googleplex.com/VVidVjFKfxT.png The fix would be to use step name on SoM to filter and match info on both side.
,
Oct 21 2016
This is a bug. Fixing.
,
Oct 21 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/77b6bad056a3427c4afe5992e72b81d33a81ccc4 commit 77b6bad056a3427c4afe5992e72b81d33a81ccc4 Author: Stephen Martinis <martiniss@chromium.org> Date: Fri Oct 21 20:20:48 2016 alerts_dispatcher: Fix findit multiple step bug BUG= 658259 Change-Id: I8f95c43f7eefc2b21b3c2a94e71e833b24b27f0a Reviewed-on: https://chromium-review.googlesource.com/401704 Reviewed-by: Sean McCullough <seanmccullough@chromium.org> Commit-Queue: Stephen Martinis <martiniss@chromium.org> [modify] https://crrev.com/77b6bad056a3427c4afe5992e72b81d33a81ccc4/go/src/infra/monitoring/analyzer/analyzer.go [modify] https://crrev.com/77b6bad056a3427c4afe5992e72b81d33a81ccc4/go/src/infra/monitoring/analyzer/analyzer_test.go
,
Oct 26 2016
Is this fixed?
,
Oct 26 2016
Yes, this is. |
||||
►
Sign in to add a comment |
||||
Comment 1 by st...@chromium.org
, Oct 21 2016