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

Issue 742481 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

paygen_au_canary: Failed to receive a successful reboot notification (update_complete) within 900 seconds

Project Member Reported by dhadd...@chromium.org, Jul 13 2017

Issue description

This is failing on a bunch of boards noticeably older ones like daisy and nyan. 

The failure is caused by the devserver_hostlog_reboot file containing no events (an empty list).

This file is generated after the update by calling update_engine_client --status and reading the events from the /api/hostlog URI.
 
Status: Assigned (was: Untriaged)
First thoughts are that the call to the hostlog URI returned nothing because the device is slow. I will try to reproduce locally and experiment with retrying the command 
Running locally on a daisy runs into the problem. The hostlog event is not updated by the time we make the call. I've added some retries. 
Project Member

Comment 4 by bugdroid1@chromium.org, Jul 14 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/82a8faf9c1d1dc7ccc60b876f157c1e6e6fed31e

commit 82a8faf9c1d1dc7ccc60b876f157c1e6e6fed31e
Author: David Haddock <dhaddock@chromium.org>
Date: Fri Jul 14 05:03:35 2017

Make CollectDevServerHostLog more robust.

On slow DUTs, CollectDevserverHostLog is returning no events because they call
is coming too quickly after the devserver has been started up. I've
added some retries to verify we have some data before continuing.

Without this the autoupdate_EndToEndTest fails with:
"Failed to receive a successful reboot notification (update_complete) within
900 seconds" because the event list is empty.

BUG= chromium:742481 
TEST=autoupdate_EndToEndTest on daisy passes with retries getting hostlog.

Change-Id: I5366950544fb0ce806846fafba1b2869c9a69329
Reviewed-on: https://chromium-review.googlesource.com/571084
Commit-Ready: David Haddock <dhaddock@chromium.org>
Tested-by: David Haddock <dhaddock@chromium.org>
Reviewed-by: Xixuan Wu <xixuan@chromium.org>

[modify] https://crrev.com/82a8faf9c1d1dc7ccc60b876f157c1e6e6fed31e/lib/auto_updater.py

Status: Fixed (was: Assigned)
Status: Started (was: Fixed)
This needs another change that is already merged but needs another dev push 
Status: Fixed (was: Started)
Cc: wuchengli@chromium.org
Status: Verified (was: Fixed)
Labels: FixedByAURewrite

Sign in to add a comment