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

Issue 679051 link

Starred by 3 users

Issue metadata

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



Sign in to add a comment

leave a "these logs were deleted" breadcrumb for logs that don't get offloaded

Project Member Reported by akes...@chromium.org, Jan 6 2017

Issue description

Example job: http://chromeos-server21.cbf.corp.google.com/afe/#tab_id=view_job&object_id=94485093

Expected behavior:
Clicking on "All logs" or "Debug logs" takes you to logs for that job.

Actual:
Clicking on those takes you to an empty directory in gs.


Reason:
We don't save jobs from passing CQ tests, under the hypothesis that people rarely need to examine these.


Ok, that sounds like a good optimization if the CQ log offloading load is a real problem. But, couldn't we at least leave a README behind explaining why those logs for that job were deleted?
 

Comment 1 Deleted

Comment 2 by dshi@chromium.org, Jan 23 2017

Labels: -current-issue
Owner: akes...@chromium.org
Project Member

Comment 4 by bugdroid1@chromium.org, Feb 4 2017

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

commit 114f2fc972054ba28027f0653c86a4432b31865f
Author: Aviv Keshet <akeshet@chromium.org>
Date: Sat Feb 04 04:01:45 2017

gs_offloader: leave a breadcrumb when we delete a bunch of results

BUG= chromium:679051 
TEST=None

Change-Id: I7bdaf164190dff1f5f83ba2e4b6797967435d3fc
Reviewed-on: https://chromium-review.googlesource.com/436529
Commit-Ready: Aviv Keshet <akeshet@chromium.org>
Tested-by: Aviv Keshet <akeshet@chromium.org>
Reviewed-by: Simran Basi <sbasi@chromium.org>

[modify] https://crrev.com/114f2fc972054ba28027f0653c86a4432b31865f/site_utils/job_directories.py

Labels: cros-infra-fixedit-q117
Status: Fixed (was: Untriaged)

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

Labels: VerifyIn-59

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

Labels: VerifyIn-60

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

Labels: VerifyIn-61

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

Status: Archived (was: Fixed)

Sign in to add a comment