Added report URLs to the dashboard.

Added report URLs to the dashboard.

Changed the test names to link to their corresponding /report page.

http://i.imgur.com/aPfbE0u.png

BUG=catapult:#3141

Review-Url: https://codereview.chromium.org/2658643005
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/8c5f79979fb59652f150d78bdb2b78c51f2d12fd

[jessimb, 2017-01-26 23:37:06]

Description was changed from

==========
Added report URLs to the dashboard.

Changed the test names to link to their corresponding /report page.

BUG=catapult:#3141
==========

to

==========
Added report URLs to the dashboard.

Changed the test names to link to their corresponding /report page.
Demo:
https://dev-jessimb-45014f56-dot-chromeperf.appspot.com/speed_releasing/Memor...

BUG=catapult:#3141
==========

[jessimb, 2017-01-27 18:39:12]

Description was changed from

==========
Added report URLs to the dashboard.

Changed the test names to link to their corresponding /report page.
Demo:
https://dev-jessimb-45014f56-dot-chromeperf.appspot.com/speed_releasing/Memor...

BUG=catapult:#3141
==========

to

==========
Added report URLs to the dashboard.

Changed the test names to link to their corresponding /report page.
Demo:
https://dev-jessimb-6ede30f7-dot-chromeperf.appspot.com/speed_releasing/Memor...

BUG=catapult:#3141
==========

[jessimb, 2017-01-27 18:51:50]

Description was changed from

==========
Added report URLs to the dashboard.

Changed the test names to link to their corresponding /report page.
Demo:
https://dev-jessimb-6ede30f7-dot-chromeperf.appspot.com/speed_releasing/Memor...

BUG=catapult:#3141
==========

to

==========
Added report URLs to the dashboard.

Changed the test names to link to their corresponding /report page.
Demo:
https://dev-jessimb-6ede30f7-dot-chromeperf.appspot.com/speed_releasing/Memor...

http://i.imgur.com/5QPdgyu.png

BUG=catapult:#3141
==========

[jessimb, 2017-02-03 17:52:56]

Description was changed from

==========
Added report URLs to the dashboard.

Changed the test names to link to their corresponding /report page.
Demo:
https://dev-jessimb-6ede30f7-dot-chromeperf.appspot.com/speed_releasing/Memor...

http://i.imgur.com/5QPdgyu.png

BUG=catapult:#3141
==========

to

==========
Added report URLs to the dashboard.

Changed the test names to link to their corresponding /report page.
Demo:
https://dev-jessimb-8396e498-dot-chromeperf.appspot.com/speed_releasing/Memor...

http://i.imgur.com/aPfbE0u.png

BUG=catapult:#3141
==========

[jessimb, 2017-02-03 18:16:10]

jessimb@chromium.org changed reviewers:
+ eakuefner@chromium.org

[jessimb, 2017-02-03 18:16:11]

PTAL
I have updated this to include the removal of other stats.

[eakuefner, 2017-02-03 18:32:04]

https://codereview.chromium.org/2658643005/diff/60001/dashboard/dashboard/ele...
File dashboard/dashboard/elements/speed-releasing-table.html (right):

https://codereview.chromium.org/2658643005/diff/60001/dashboard/dashboard/ele...
dashboard/dashboard/elements/speed-releasing-table.html:132: 
oops?

https://codereview.chromium.org/2658643005/diff/60001/dashboard/dashboard/spe...
File dashboard/dashboard/speed_releasing.py (right):

https://codereview.chromium.org/2658643005/diff/60001/dashboard/dashboard/spe...
dashboard/dashboard/speed_releasing.py:166: if len(test.split('/')) > 2:
can you explain a bit about this logic? why do we want to check all by default
but do this other thing? maybe some comments here?

also, you should create test_parts and bot_parts so that you only have to do the
split once.

[jessimb, 2017-02-03 19:42:44]

PTAL

https://codereview.chromium.org/2658643005/diff/60001/dashboard/dashboard/ele...
File dashboard/dashboard/elements/speed-releasing-table.html (right):

https://codereview.chromium.org/2658643005/diff/60001/dashboard/dashboard/ele...
dashboard/dashboard/elements/speed-releasing-table.html:132: 
On 2017/02/03 at 18:32:04, eakuefner wrote:
> oops?

Merge conflicts :(

https://codereview.chromium.org/2658643005/diff/60001/dashboard/dashboard/spe...
File dashboard/dashboard/speed_releasing.py (right):

https://codereview.chromium.org/2658643005/diff/60001/dashboard/dashboard/spe...
dashboard/dashboard/speed_releasing.py:166: if len(test.split('/')) > 2:
On 2017/02/03 at 18:32:04, eakuefner wrote:
> can you explain a bit about this logic? why do we want to check all by default
but do this other thing? maybe some comments here?
> 
> also, you should create test_parts and bot_parts so that you only have to do
the split once.

So I was originally following the logic from the Chrome Health dashboard, where
what should be checked is foreground and background. Instead of going after the
2nd /, I think the correct thing here is the to check what is after the last /.
Will add comments to help explain this.

[eakuefner, 2017-02-21 22:29:18]

eakuefner@chromium.org changed reviewers:
+ sullivan@chromium.org

[eakuefner, 2017-02-21 22:29:19]

+Annie

lgtm -- please remove the demo link from your description as the demo doesn't
seem to work anymore.

[jessimb, 2017-02-21 22:30:11]

Description was changed from

==========
Added report URLs to the dashboard.

Changed the test names to link to their corresponding /report page.
Demo:
https://dev-jessimb-8396e498-dot-chromeperf.appspot.com/speed_releasing/Memor...

http://i.imgur.com/aPfbE0u.png

BUG=catapult:#3141
==========

to

==========
Added report URLs to the dashboard.

Changed the test names to link to their corresponding /report page.

http://i.imgur.com/aPfbE0u.png

BUG=catapult:#3141
==========

[sullivan, 2017-02-22 15:31:31]

lgtm

[jessimb, 2017-02-22 17:15:59]

The CQ bit was checked by jessimb@chromium.org

[commit-bot: I haz the power, 2017-02-22 17:16:08]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-22 17:17:17]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-22 17:17:18]

Try jobs failed on following builders:
  Catapult Android Tryserver on master.tryserver.client.catapult (JOB_FAILED,
https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20An...)
  Catapult Linux Tryserver on master.tryserver.client.catapult (JOB_FAILED,
https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Li...)
  Catapult Presubmit on master.tryserver.client.catapult (JOB_FAILED,
https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Pr...)

[jessimb, 2017-02-22 18:42:47]

The CQ bit was checked by jessimb@chromium.org

[jessimb, 2017-02-22 18:42:48]

The patchset sent to the CQ was uploaded after l-g-t-m from
eakuefner@chromium.org, sullivan@chromium.org
Link to the patchset: https://codereview.chromium.org/2658643005/#ps120001
(title: "resolving merge conficts")

[commit-bot: I haz the power, 2017-02-22 18:42:53]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-22 19:11:46]

CQ is committing da patch.

Bot data: {"patchset_id": 120001, "attempt_start_ts": 1487788967579430,
"parent_rev": "d0bd70cff902c124553c4658b439f9e528487a65", "commit_rev":
"8c5f79979fb59652f150d78bdb2b78c51f2d12fd"}

[commit-bot: I haz the power, 2017-02-22 19:11:49]

Description was changed from

==========
Added report URLs to the dashboard.

Changed the test names to link to their corresponding /report page.

http://i.imgur.com/aPfbE0u.png

BUG=catapult:#3141
==========

to

==========
Added report URLs to the dashboard.

Changed the test names to link to their corresponding /report page.

http://i.imgur.com/aPfbE0u.png

BUG=catapult:#3141

Review-Url: https://codereview.chromium.org/2658643005
Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2017-02-22 19:11:50]

Committed patchset #7 (id:120001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...