Broken links for closures in tree status apps |
|||||
Issue descriptionStarting around March 17, tree status apps don't have links to the failed build cycles anymore as shown http://chromium-status.appspot.com/?limit=250 Same for infra/v8 tree status app. But a link is very convenient to open the bot to investigate the failure.
,
Mar 24 2017
,
Mar 24 2017
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/f7cf1ac1311e63cb199c4ca5d606e6dab5a97b9f commit f7cf1ac1311e63cb199c4ca5d606e6dab5a97b9f Author: Sean McCullough <seanmccullough@chromium.org> Date: Fri Mar 24 21:28:43 2017 [*-status.appspot.com] try to be smarter about linking builds Bug: 704974 Change-Id: If168ab26ad38cbbd14259c6a1a9e11bee6a79e3e Reviewed-on: https://chromium-review.googlesource.com/459228 Reviewed-by: Aaron Gable <agable@chromium.org> Commit-Queue: Sean McCullough <seanmccullough@chromium.org> [modify] https://crrev.com/f7cf1ac1311e63cb199c4ca5d606e6dab5a97b9f/appengine/chromium_status/appengine_module/chromium_status/status.py
,
Apr 4 2017
,
May 12 2017
Looks like this did not fix the app. I've deployed newest version to chromium-status.appspot.com, but this didn't help.
,
Feb 24 2018
(Lowering priority since I don't think this is P1) But if you disagree, feel free to raise the priority again. My long-term solution to this will probably be to create a geenric, shared free text parsing element for ChOpsUI and have Polymer frontends use that element to format user inputs. I do think there's value in keeping the random free text parsing since it's convenient, especially for inherently unstructured user input. But it should be automatically tested and standardized (so users can learn our schemes). |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by seanmccullough@chromium.org
, Mar 24 2017