New issue
Advanced search Search tips

Issue 873331 link

Starred by 1 user
Status: Fixed
Owner:
dtu@chromium.org
Closed: Aug 14
Cc:
simonhatch@chromium.org
st...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

[📍] When pressing re-run button: "could not convert string to float: None"

Project Member Reported by dtu@chromium.org, Aug 10 
I think this is related to  issue 873022 .

Previously, perf dashboard was kicking off jobs with comparison_magnitude set to the string 'None'. Now, that value is no longer valid. comparison_magnitude must be a float. So, re-kicking those old jobs will give an error.

Short term solution is to add a check somewhere in the form or backend that ignores comparison_magnitude == 'None'.
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 10 

The following revision refers to this bug:
  https://chromium.googlesource.com/catapult/+/21cd5a309631b78d6deedc1ca56ca83ead9fbadd

commit 21cd5a309631b78d6deedc1ca56ca83ead9fbadd
Author: Dave Tu <dtu@chromium.org>
Date: Fri Aug 10 22:50:54 2018

[pinpoint] Ignore comparison_magnitude == 'None' for reruns.

Perf dashboard has been sending jobs with comparison_magnitude == 'None'
(The string containing 'None'.) This is no longer permitted in the API.
This is a workaround that lets us re-run jobs with the invalid argument.

Bug:  chromium:873331 
Change-Id: I69ae79ec6317f86e9411f519856b294741d0115d
Reviewed-on: https://chromium-review.googlesource.com/1171793
Reviewed-by: Annie Sullivan <sullivan@chromium.org>
Commit-Queue: David Tu <dtu@chromium.org>

[modify] https://crrev.com/21cd5a309631b78d6deedc1ca56ca83ead9fbadd/dashboard/dashboard/pinpoint/elements/job-page/rerun-job-fab.html
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 11 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/24be12346d5a4ec8ebcc2c0615add14d150496e7

commit 24be12346d5a4ec8ebcc2c0615add14d150496e7
Author: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Date: Sat Aug 11 00:34:54 2018

Roll src/third_party/catapult 5a4110982f31..21cd5a309631 (1 commits)

https://chromium.googlesource.com/catapult.git/+log/5a4110982f31..21cd5a309631


git log 5a4110982f31..21cd5a309631 --date=short --no-merges --format='%ad %ae %s'
2018-08-10 dtu@chromium.org [pinpoint] Ignore comparison_magnitude == 'None' for reruns.


Created with:
  gclient setdep -r src/third_party/catapult@21cd5a309631

The AutoRoll server is located here: https://catapult-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel

BUG= chromium:873331 
TBR=sullivan@chromium.org

Change-Id: Ie322f1501b9f5411006fd4d603edf90941097a93
Reviewed-on: https://chromium-review.googlesource.com/1171404
Reviewed-by: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#582399}
[modify] https://crrev.com/24be12346d5a4ec8ebcc2c0615add14d150496e7/DEPS

Comment 3 by nednguyen@chromium.org, Aug 13

Blocking: 873724

Comment 4 by dtu@chromium.org, Aug 14

Status: Fixed (was: Assigned)

Comment 5 by dtu@chromium.org, Aug 14

Blocking: -873724
►

Sign in to add a comment