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

Issue 782837 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Histogram update scripts no longer work due to missing argument

Project Member Reported by rsleevi@chromium.org, Nov 8 2017

Issue description

https://chromium.googlesource.com/chromium/src/+/d7690ddedbda1f141d1eb98bbd236ab192ecca8f introduced an additional argument to UpdateHistogramFromDict, which takes caller_script_name as part of the annotations.

However, this fourth argument breaks the following scripts:
update_gpu_driver_bug_workaround_entries.py
update_net_error_codes.py
update_use_counter_css.py


 
Apologies, update_use_counter_css.py actually works
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 8 2017

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

commit c4b941586ad6de249c8c371593a2ef9ccae90173
Author: Ryan Sleevi <rsleevi@chromium.org>
Date: Wed Nov 08 22:47:33 2017

Update the net error code and GPU driver bug workaround histograms

Update the network error codes and GPU driver bug workarounds for the
current release. This also updates the scripts to pass in the calling
script argument, introduced in d7690ddedbda1f141d1eb98bbd236ab192ecca8f

BUG= 782837 

Change-Id: Ia9f1dc7f9136b7419e85dafe395d0828fd4569fd
Reviewed-on: https://chromium-review.googlesource.com/758933
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Commit-Queue: Ilya Sherman <isherman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#514980}
[modify] https://crrev.com/c4b941586ad6de249c8c371593a2ef9ccae90173/tools/metrics/histograms/enums.xml
[modify] https://crrev.com/c4b941586ad6de249c8c371593a2ef9ccae90173/tools/metrics/histograms/update_gpu_driver_bug_workaround_entries.py
[modify] https://crrev.com/c4b941586ad6de249c8c371593a2ef9ccae90173/tools/metrics/histograms/update_net_error_codes.py

Labels: M-64
Status: Verified (was: Started)

Sign in to add a comment