New issue
Advanced search Search tips

Issue 608580 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Feb 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: ----

Blocked on:
issue 608583
issue 613765
issue 613769

Blocking:
issue 627269
issue 635639



Sign in to add a comment

Overhaul devserver: from pull to push

Project Member Reported by xixuan@chromium.org, May 3 2016

Issue description

Currently when we need to update a CrOS DUT, we need to call devserver to stage artifacts, call DUT to fetch artifacts from devserver, and let DUT execute auto-update. The process is too complicated to debug, and easily down due to network congestion/flake. 

We want to change the workflow to: let devserver call DUT to update. The DUT will never pull devserver to update. Instead, devserver will push the payload to DUT and call it to update. 

The advantages include:
1. Reduce errors caused by network congestion/flake: 
remove the two most flake rpcs: cautotest calls DUT to auto-update, and DUT pulls devserver to transfer payload. Change them with a much reliable rpc: devserver pushes payload to DUT, and let DUT auto-update.

2. Easy to debug. Move every steps, such as stage, transfer payloads, and execute auto-update to devserver, and log relative debugging errors.

3. Leave room to modify devserver's form. 

4. Leave room to integrate the auto-update logics for all devices and images to this framework.

 
Blockedon: 608583

Comment 2 by xixuan@chromium.org, May 21 2016

Blockedon: 613765

Comment 3 by xixuan@chromium.org, May 21 2016

Blockedon: 613769
Components: Infra>Client>ChromeOS
Components: -Infra

Comment 6 by xixuan@chromium.org, Jul 11 2016

Blocking: 627269
Blocking: 635639

Comment 8 by sosa@chromium.org, Aug 9 2016

Labels: -Pri-2 Pri-1
Project Member

Comment 9 by bugdroid1@chromium.org, Sep 9 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/chromeos-admin/+/ce8c8708e1e19085e0c7d9481268d44d4d08f262

commit ce8c8708e1e19085e0c7d9481268d44d4d08f262
Author: xixuan <xixuan@google.com>
Date: Tue Aug 09 19:14:01 2016

Project Member

Comment 10 by bugdroid1@chromium.org, Sep 15 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/chromeos-admin/+/f08c94088d6021a22fbea015605cf81ded3837df

commit f08c94088d6021a22fbea015605cf81ded3837df
Author: xixuan <xixuan@chromium.org>
Date: Thu Sep 15 18:22:56 2016

Project Member

Comment 11 by bugdroid1@chromium.org, Sep 22 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/chromeos-admin/+/7587c94fd36f21c46750e0ac9be5e9e5c302de8a

commit 7587c94fd36f21c46750e0ac9be5e9e5c302de8a
Author: xixuan <xixuan@chromium.org>
Date: Thu Sep 22 21:00:51 2016

Project Member

Comment 12 by bugdroid1@chromium.org, Oct 5 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/chromeos-admin/+/776d5619d8c98bcdcec9b9262655912188892f98

commit 776d5619d8c98bcdcec9b9262655912188892f98
Author: xixuan <xixuan@chromium.org>
Date: Wed Oct 05 20:15:31 2016

Status: Fixed (was: Assigned)
Status: Verified (was: Fixed)

Sign in to add a comment