New issue
Advanced search Search tips

Issue 824146 link

Starred by 1 user
Status: Fixed
Owner:
xixuan@chromium.org
Closed: Apr 2018
Cc:
akes...@chromium.org
nya@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Milo links to flake-dashboard and test-history go to deprecated, empty wmatrix pages

Project Member Reported by jamescook@chromium.org, Mar 21 2018 
* Put a change in the CQ, have the CQ run fail (e.g. https://chromium-review.googlesource.com/c/chromiumos/platform/tast-tests/+/972154 )
* Go to the build results: https://luci-milo.appspot.com/buildbot/chromeos/guado_moblab-paladin/8949#
* Be suspicious the breakage isn't your fault
* Click "Flake-dashboard" and "Test-history"

https://screenshot.googleplex.com/g7Eg56cPntm

These take you to:
https://wmatrix.googleplex.com/retry_teststats/?days_back=30&tests=moblab_RunSuite
and
https://wmatrix.googleplex.com/unfiltered?hide_missing=True&tests=moblab_RunSuite

which are empty pages.

As a naive Chrome OS developer, I don't know if we still have a flake dashboard. I'm also not certain what the appropriate Stainless query is for the test history.

Ideally these links would work. Failing that, they should be removed.

Comment 1 by akes...@chromium.org, Mar 21 2018

Cc: nya@chromium.org
Owner: xixuan@chromium.org
Status: Assigned (was: Untriaged)
Agreed, this are both deprecated.

There's a more useful set of dashboards on stainless that we could link to instead of wmatrix. (xixuan@ can we link to that instead today?)

In the long term, and new flake dashboard based on the same stainless back end is in the works.

In the meantime, let's delete links to obsolete services.

Comment 2 by xixuan@chromium.org, Mar 21 2018

Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 26 2018 

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

commit 1729fca0ad10f693fdc17bd0bb9497482dfa97c4
Author: Xixuan Wu <xixuan@chromium.org>
Date: Mon Mar 26 21:41:32 2018

autotest: Replace test_history url with stainless.

This CL also removes flake_dashboard url. It will be added back when new
flake dashboard is ready.

BUG= chromium:824146 
TEST=Ran unittest.

Change-Id: I4983166a6a797ae6941540ef6430e2f692b5649f
Reviewed-on: https://chromium-review.googlesource.com/974509
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Xixuan Wu <xixuan@chromium.org>

[modify] https://crrev.com/1729fca0ad10f693fdc17bd0bb9497482dfa97c4/site_utils/run_suite.py
[modify] https://crrev.com/1729fca0ad10f693fdc17bd0bb9497482dfa97c4/server/cros/dynamic_suite/reporting_unittest.py
[modify] https://crrev.com/1729fca0ad10f693fdc17bd0bb9497482dfa97c4/server/cros/dynamic_suite/reporting_utils.py
[modify] https://crrev.com/1729fca0ad10f693fdc17bd0bb9497482dfa97c4/global_config.ini

Comment 4 by xixuan@chromium.org, Apr 17 2018

Status: Fixed (was: Started)
►

Sign in to add a comment