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

Issue 604039 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Apr 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

autofiled bug links on waterfall should explicitly say "Auto-bug: " , and should also still link separately to cautotest logs

Project Member Reported by akes...@chromium.org, Apr 15 2016

Issue description

When HWTest failures cause auto-filed bugs to be filed or pinged, run_suite will emit a STEP_LINK annotation which causes the waterfall to link to that bug.

However, the link test doesn't make it obvious that it is a link to the bug. These links should start with the text "Auto-bug" to make it obvious.

Also, it seems like in cases where we link to an autofiled bug, we suppress the link to the cautotest log. We should *always* link to the cautotest log too, even if we are linking to an autobug. Perhaps with some addtional text that makes explicit that is it a test log link, like "Test logs: "

Example: https://uberchromegw.corp.google.com/i/chromeos/builders/falco-chrome-pfq/builds/4150
 

Comment 1 by autumn@chromium.org, Apr 18 2016

Labels: -current-issue
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 22 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/bd911bd8f29faa2a6fd161dbb7f2d05fc56a2900

commit bd911bd8f29faa2a6fd161dbb7f2d05fc56a2900
Author: Ningning Xia <nxia@chromium.org>
Date: Tue Apr 19 21:06:03 2016

Add [Auto-Bug]/[Test-Logs] prefix to hwtest cbuildbot links

Add Auto-Bug prefix to the anchor of bug links. Add Test-Logs to the
anchor of log links. Also link to cautotest logs when an auto bug is
filed and linked.

BUG= chromium:604039 
TEST=run_suite_unittest

Change-Id: Idf6030164055c3814a6acff4ba2841061bdf337b
Reviewed-on: https://chromium-review.googlesource.com/339712
Commit-Ready: Ningning Xia <nxia@chromium.org>
Tested-by: Ningning Xia <nxia@chromium.org>
Reviewed-by: Aviv Keshet <akeshet@chromium.org>

[modify] https://crrev.com/bd911bd8f29faa2a6fd161dbb7f2d05fc56a2900/site_utils/run_suite.py
[modify] https://crrev.com/bd911bd8f29faa2a6fd161dbb7f2d05fc56a2900/site_utils/run_suite_unittest.py

Comment 3 by nxia@chromium.org, Apr 23 2016

Status: Fixed (was: Assigned)

Comment 4 by benhenry@google.com, Apr 27 2016

Components: Infra>Client>ChromeOS
Labels: -Infra-ChromeOS
Bulk verified
Status: Verified (was: Fixed)
bulk verified

Sign in to add a comment