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

Issue 632490 link

Starred by 4 users

Issue metadata

Status: Archived
Owner:
Last visit > 30 days ago
Closed: Aug 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Only upload CTS/GTS results for release builds

Project Member Reported by nxia@chromium.org, Jul 28 2016

Issue description

GE team is only interested in non-trybot and *-release CTS/GTS results. 

Request for only uploading test results in related release builds. 


 

Comment 1 by nxia@chromium.org, Jul 28 2016

Status: Started (was: Untriaged)
Labels: -OS-Linux OS-Chrome
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 30 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/21922c807733cab1a86096500304aa8ae68bd897

commit 21922c807733cab1a86096500304aa8ae68bd897
Author: Ningning Xia <nxia@chromium.org>
Date: Fri Jul 29 18:03:15 2016

Only offload CTS/GTS results for release builds.

Only offload results for release builds, not including pfq, paladin or
trybots builds.

BUG= chromium:632490 
TEST=unit_test;run_gs_offloader

Change-Id: I93107c773c242f0193447148a37f47491e5af651
Reviewed-on: https://chromium-review.googlesource.com/364720
Commit-Ready: Ningning Xia <nxia@chromium.org>
Tested-by: Ningning Xia <nxia@chromium.org>
Reviewed-by: Jason Kusuma <jkusuma@chromium.org>
Reviewed-by: Dan Shi <dshi@google.com>

[modify] https://crrev.com/21922c807733cab1a86096500304aa8ae68bd897/site_utils/gs_offloader_unittest.py
[modify] https://crrev.com/21922c807733cab1a86096500304aa8ae68bd897/site_utils/gs_offloader.py

Cc: rohi...@chromium.org dshi@chromium.org leecy@chromium.org
Could we also do a suite filter for suite = 'arc-cts' only?
It seems that there are a few results from bvt-cq with suite = 'arc-bvt-cq' or suite = 'bvt-cq' that still ends up in the gs bucket e.g. 
gs://chromeos-cts-results/cheets_CTS.android.core.tests.libcore.package.harmony_java_math/73057362-chromeos-test_2016.08.12_16.01.09/testResult.xml.gz
gs://chromeos-cts-results/cheets_CTS.android.core.tests.libcore.package.harmony_java_math/73057258-chromeos-test_2016.08.12_15.58.32/testResult.xml.gz
gs://chromeos-cts-results/cheets_CTS.android.core.tests.libcore.package.harmony_java_math/73044245-chromeos-test_2016.08.12_06.33.54/testResult.xml.gz

Comment 5 by nxia@chromium.org, Aug 13 2016

moved comment #4 requirement to crbug/637498

Comment 6 by nxia@chromium.org, Aug 13 2016

Status: Fixed (was: Started)
Labels: VerifyIn-54

Comment 8 by dchan@chromium.org, Oct 7 2016

Labels: VerifyIn-55

Comment 9 by dchan@google.com, Nov 19 2016

Labels: VerifyIn-56

Comment 10 by dchan@google.com, Jan 21 2017

Labels: VerifyIn-57

Comment 11 by dchan@google.com, Mar 4 2017

Labels: VerifyIn-58

Comment 12 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59

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

Labels: VerifyIn-60
Labels: VerifyIn-61

Comment 15 by dchan@chromium.org, Oct 14 2017

Status: Archived (was: Fixed)

Sign in to add a comment