[Findit] Flake Analyzer - Analysis made it to completion but an error is reported |
|||
Issue descriptionhttps://findit-for-me.appspot.com/waterfall/flake?key=ag9zfmZpbmRpdC1mb3ItbWVypwELEhdNYXN0ZXJGbGFrZUFuYWx5c2lzUm9vdCJxY2hyb21pdW0ubWVtb3J5L0xpbnV4IFRTYW4gVGVzdHMvMTk5ODgvY29udGVudF91bml0dGVzdHMvUW5KdmQzTnBibWRFWVhSaFVtVnRiM1psY2tsdGNHeFVaWE4wTGsxMWJIUnBjR3hsVkdGemEzTT0MCxITTWFzdGVyRmxha2VBbmFseXNpcxgBDA This analysis worked well, but in the end it reports there was an error when there was none. The UI even surfaces "None" as the error yet sets the status to error.
,
Apr 10 2018
,
Apr 11 2018
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/22f62d09326c59d67fabb1fa1db0fdf30e575359 commit 22f62d09326c59d67fabb1fa1db0fdf30e575359 Author: Brandon Wylie <wylieb@chromium.org> Date: Wed Apr 11 18:24:25 2018 [Findit] Flake Analyzer - Support a single analysis status in the UI The old pipelines had three different analyses (1) regression range (or build-level) status. (2) Culprit analysis (or commit-level) status. (3) Heuristic analysis status. The new pipelines have only one status, which is the status for the entire analysis. This change instruments this in the ui and also on the handler. Notable changes include: * Support for old analyses (that will assumed to all be completed or ended in error since the new pipelines are 100% running now) by computing a single analyses server-side. * Changing the frontend polymer components to support this. * Changing the template to accept the new status parameter and passing that through to polymer element flake-analysis-info. Bug: 830655 Change-Id: I11162164582ce13dd6f4164aa6ee50d968bba680 Reviewed-on: https://chromium-review.googlesource.com/1006478 Commit-Queue: Brandon Wylie <wylieb@chromium.org> Reviewed-by: Shuotao Gao <stgao@chromium.org> [modify] https://crrev.com/22f62d09326c59d67fabb1fa1db0fdf30e575359/appengine/findit/ui/elements/analysis/flake/flake-analysis-info-demo.html [modify] https://crrev.com/22f62d09326c59d67fabb1fa1db0fdf30e575359/appengine/findit/ui/elements/analysis/flake/flake-analysis-info.html [modify] https://crrev.com/22f62d09326c59d67fabb1fa1db0fdf30e575359/appengine/findit/handlers/flake/check_flake.py [modify] https://crrev.com/22f62d09326c59d67fabb1fa1db0fdf30e575359/appengine/findit/handlers/flake/test/check_flake_test.py [modify] https://crrev.com/22f62d09326c59d67fabb1fa1db0fdf30e575359/appengine/findit/templates/flake/result.html
,
Apr 12 2018
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/bbb50f8f027b0e69b6c68783348fbba5ab9394e9 commit bbb50f8f027b0e69b6c68783348fbba5ab9394e9 Author: Brandon Wylie <wylieb@chromium.org> Date: Thu Apr 12 00:18:18 2018 [Findit] Flake Analyzer - Support single analysis status for list-flakes Analyses statuses have been condensed down to one status for the new pipelines. This change will support this and also the old format. Bug: 830655 Change-Id: I7923062a88a59ded5e745e6139ad68fe75280d5f Reviewed-on: https://chromium-review.googlesource.com/1008965 Commit-Queue: Brandon Wylie <wylieb@chromium.org> Reviewed-by: Shuotao Gao <stgao@chromium.org> [modify] https://crrev.com/bbb50f8f027b0e69b6c68783348fbba5ab9394e9/appengine/findit/templates/flake/dashboard.html [modify] https://crrev.com/bbb50f8f027b0e69b6c68783348fbba5ab9394e9/appengine/findit/handlers/flake/list_flakes.py [modify] https://crrev.com/bbb50f8f027b0e69b6c68783348fbba5ab9394e9/appengine/findit/handlers/flake/test/list_flakes_test.py
,
Apr 12 2018
|
|||
►
Sign in to add a comment |
|||
Comment 1 by lijeffrey@chromium.org
, Apr 9 2018Status: Assigned (was: Available)