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

Issue 655520 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: ----
Type: ----



Sign in to add a comment

bots no longer uploading to flakiness dashboard

Project Member Reported by stip@chromium.org, Oct 13 2016

Issue description

Ooops, sorry. My fix is in https://codereview.chromium.org/2409993005
Project Member

Comment 2 by bugdroid1@chromium.org, Oct 13 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build.git/+/1a9df62478f43484695e00ed6a808673461c5736

commit 1a9df62478f43484695e00ed6a808673461c5736
Author: nednguyen <nednguyen@google.com>
Date: Thu Oct 13 10:36:15 2016

Rename --input-gtest-json flag to --input-json flag

This is for generalizing the logic of uploading test results
to support other format such as JSON Test Result format.

BUG= 655520 , 649762 

Review-Url: https://codereview.chromium.org/2409993005

[modify] https://crrev.com/1a9df62478f43484695e00ed6a808673461c5736/scripts/slave/recipe_modules/test_results/resources/upload_test_results.py

Project Member

Comment 3 by bugdroid1@chromium.org, Oct 13 2016

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

commit e1068b5b44eace8d671ac9b8147dc95dbb77e11c
Author: recipe-roller <recipe-roller@chromium.org>
Date: Thu Oct 13 11:28:17 2016

Roll recipe dependencies (trivial).

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/1a9df62478f43484695e00ed6a808673461c5736 Rename --input-gtest-json flag to --input-json flag (nednguyen@google.com)

TBR=martiniss@chromium.org,phajdan.jr@chromium.org
BUG= 649762 , 655520 

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://codereview.chromium.org/2415923002

[modify] https://crrev.com/e1068b5b44eace8d671ac9b8147dc95dbb77e11c/infra/config/recipes.cfg

Owner: nedngu...@google.com
Status: Fixed (was: Untriaged)
This is fixed: https://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_ng/builds/310251

Comment 5 by ojan@chromium.org, Oct 13 2016

Cc: andyb...@chromium.org
Owner: estaab@chromium.org
estaab: Is there a bug to track to add monitoring for this? I'm not sure how stip discovered this.

Comment 6 by estaab@chromium.org, Oct 13 2016

Pretty sure stip discovered this when looking at a try run since the builds became littered with yellow steps.

We have a dreadfully old bug here: http://crbug.com/400458
I'll update it with new info about this regression.

It looks like the metrics we added didn't catch this. We need to break the requests down so we can see just uploads and alert on those.

https://viceroy.corp.google.com/chrome_infra/Appengine/test_results
Project Member

Comment 7 by bugdroid1@chromium.org, Oct 14 2016

Sign in to add a comment