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

Issue 601730 link

Starred by 4 users

Issue metadata

Status: Verified
Merged: issue 602208
Owner:
Last visit > 30 days ago
Closed: May 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Screenshot upload should retry on error

Project Member Reported by atwilson@chromium.org, Apr 8 2016

Issue description

If there's an error uploading screenshots for monitoring purposes, we should retry a few times - it should be possible to do this just by updating kMaxRetries in upload_job_impl.cc and updating the tests.


 
Cc: jinzhang@chromium.org pbond@chromium.org
Mergedinto: 602208
Status: Duplicate (was: Untriaged)
Cc: hunyadym@chromium.org
 Issue 602208  has been merged into this issue.
Status: Assigned (was: Duplicate)
duping bugs on eachother seems bad - reopening this one since it was first.
We also should wait 20 seconds between each retry rather than retrying immediately - that way we are resilient to transient network failures. It's OK to add this as a separate CL from enabling retries.
Cc: sduraisamy@chromium.org
Friendly ping for an update.
Status: Started (was: Assigned)
See CL at https://codereview.chromium.org/1875443003/, feedback is welcome.
Cc: ken...@chromium.org
Thanks Márton. Any update on the CL? It seems there's no activity for a while. Today, Costco (one of our customer) reported that their screenshot commands are always failing. I believe fixing this issue would improve user experiences. Can you please prioritize this work item? Thanks.
Status: Fixed (was: Started)
Components: UI>Shell>Kiosk
Labels: M-53 OS-Chrome
Labels: -M-53 M-52
Cc: mlight@chromium.org harpreet@chromium.org

Comment 15 by mlight@google.com, Aug 15 2016

Status: Verified (was: Fixed)
Verified in M52-Stable, build 8350.69.0
Also verified in M53-Beta, build 8530.57.0

Sign in to add a comment