Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(58)

Issue 1885033002: BuildBot: Add anchors for unbound aliases. (Closed)

Created:
4 years, 8 months ago by dnj
Modified:
4 years, 7 months ago
CC:
chromium-reviews, infra-reviews+build_chromium.org
Target Ref:
refs/heads/master
Project:
build
Visibility:
Public.

Description

BuildBot: Add anchors for unbound aliases. Currently, aliases only attach themselves to logs/links that reference them. If those logs/links disappear, the alias is not shown. This change makes it so that aliases without log/link anchors will generate link anchors. This solves the problem of log aliase disappearing when the logs are pruned from the master. BUG=chromium:602384 TEST=local - Created aliased builds, deleted logs, still render. R=estaab@chromium.org, iannucci@chromium.org, martiniss@chromium.org, sergiyb@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=300424

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+53 lines, -13 lines) Patch
M masters/master.chromium.perf.fyi/templates/build.html View 1 chunk +5 lines, -1 line 0 comments Download
M masters/master.chromium/templates/build.html View 1 chunk +5 lines, -1 line 1 comment Download
M masters/master.chromiumos/templates/build.html View 1 chunk +5 lines, -1 line 0 comments Download
M masters/master.tryserver.blink/templates/build.html View 1 chunk +5 lines, -1 line 0 comments Download
M masters/master.tryserver.chromium.linux/templates/build.html View 1 chunk +5 lines, -1 line 0 comments Download
M masters/master.tryserver.chromium.perf/templates/build.html View 1 chunk +5 lines, -1 line 0 comments Download
M third_party/buildbot_8_4p1/buildbot/status/web/build.py View 2 chunks +23 lines, -7 lines 1 comment Download

Messages

Total messages: 7 (2 generated)
dnj
4 years, 8 months ago (2016-04-13 15:24:44 UTC) #1
dnj
https://codereview.chromium.org/1885033002/diff/1/masters/master.chromium/templates/build.html File masters/master.chromium/templates/build.html (right): https://codereview.chromium.org/1885033002/diff/1/masters/master.chromium/templates/build.html#newcode154 masters/master.chromium/templates/build.html:154: <span>{{ l.name }}</span> (Will never be hit for scenario ...
4 years, 8 months ago (2016-04-13 15:40:00 UTC) #2
Sergiy Byelozyorov
lgtm
4 years, 8 months ago (2016-04-13 17:31:14 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1885033002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1885033002/1
4 years, 7 months ago (2016-05-03 04:40:45 UTC) #5
commit-bot: I haz the power
4 years, 7 months ago (2016-05-03 04:45:03 UTC) #7
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=300424

Powered by Google App Engine
This is Rietveld 408576698