New issue
Advanced search Search tips

Issue 705183 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Tree Open/Closed Stats metrics are broken.

Project Member Reported by dgarr...@chromium.org, Mar 25 2017

Issue description

This link:

https://chromiumos-status.appspot.com/status_viewer?curView=stats&startTime=TODAY&numDays=7

Is often wrong. It currently shows that the tree was never closed, when it was this week.

Is also sometimes generates negative percentages, and other numbers that are clearly wrong.

Finally, if we modify it, we should provide fetchable formatted data suitable for import directly into summarize_build_stats.
 

Comment 1 by aut...@google.com, Mar 28 2017

Owner: akes...@chromium.org
Aviv - who owns this page? Do we care enough to fix it? Or should we remove from the weekly report? 
Cc: akes...@chromium.org
Owner: xixuan@chromium.org
No idea who owns it, and nobody cares enough to fix it I think. Let's remove it from report.

This is a 2-line change to the summarize_build_scripts template.
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 29 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/f747e79df252f29bf476a8e04d04cc9e856ed3b4

commit f747e79df252f29bf476a8e04d04cc9e856ed3b4
Author: xixuan <xixuan@chromium.org>
Date: Wed Mar 29 18:42:19 2017

chromite: Remove 'tree status report' in summarize_build_stats

This CL delete the 'tree status report' part in summarize_build_stats.

BUG= chromium:705183 
TEST=Run bin/summarize_build_stats.

Change-Id: Ic2eecdbd78ec63fb93e2102af4bd1110bd0d642c
Reviewed-on: https://chromium-review.googlesource.com/462085
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>

[modify] https://crrev.com/f747e79df252f29bf476a8e04d04cc9e856ed3b4/scripts/summarize_build_stats.py

Comment 4 by xixuan@chromium.org, Mar 29 2017

Status: Fixed (was: Untriaged)

Comment 5 by dchan@google.com, May 30 2017

Labels: VerifyIn-60

Comment 6 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

Comment 7 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment