New issue
Advanced search Search tips

Issue 868897 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 14
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

CopyProgress is spamming the logs and incorrectly(?) reporting error

Project Member Reported by zentaro@chromium.org, Jul 30

Issue description

Repro:

Do a recursive copy of several hundred/thousand files.

The logs get spammed with Errors about copy progress with error number 51 IIRC.

I suspect this is the have_more_ask_again code. This should be filtered out of the error logging.
 
Status: Started (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 10

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/35f3ca6df8747d454c1f3430df5b7788089d5f49

commit 35f3ca6df8747d454c1f3430df5b7788089d5f49
Author: Jimmy Gong <jimmyxgong@google.com>
Date: Fri Aug 10 05:04:36 2018

smbprovider: Fix ContinueCopy log spam bug

- Previously, ContinueCopy would log everytime it performs a new
ContinueCopy. This fix correctly logs any unexpected errors.

BUG= chromium:868897 
TEST=emerges

Change-Id: I55f5d19894acfcffc53ba5c977e792ffe0c54ce8
Reviewed-on: https://chromium-review.googlesource.com/1166336
Commit-Ready: jimmy gong <jimmyxgong@chromium.org>
Tested-by: jimmy gong <jimmyxgong@chromium.org>
Reviewed-by: Zentaro Kavanagh <zentaro@chromium.org>

[modify] https://crrev.com/35f3ca6df8747d454c1f3430df5b7788089d5f49/smbprovider/smbprovider.cc

Status: Fixed (was: Started)

Sign in to add a comment