New issue
Advanced search Search tips

Issue 641549 link

Starred by 2 users

Issue metadata

Status: Archived
Owner:
Closed: Oct 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: ----

Blocking:
issue 635288



Sign in to add a comment

Move moblab-only rpcs to moblab_rpc_interface

Project Member Reported by xixuan@chromium.org, Aug 26 2016

Issue description

As one step of bug:635288, this bug is for tracking moving moblab-related RPCs to new module: moblab_rpc_interface.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 8 2016

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

commit ba232a36e058b58188a662647eeaa52c2b76a945
Author: xixuan <xixuan@chromium.org>
Date: Fri Aug 26 00:01:59 2016

Autotest: Move moblab-related RPC to moblab_rpc_interface.py.

This CL moves all RPCs that can only be run on moblab to moblab_rpc_interface.
Also move related unittests to moblab_rpc_interface_unittest.

BUG= chromium:641549 
TEST=cros flash to a local moblab, upload boto key, then run a suite on it.

Change-Id: I8246282a4c7250350dd9a4e9043eaeba293ccb34
Reviewed-on: https://chromium-review.googlesource.com/377418
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Fang Deng <fdeng@chromium.org>
Reviewed-by: Dan Shi <dshi@google.com>

[add] https://crrev.com/ba232a36e058b58188a662647eeaa52c2b76a945/frontend/afe/moblab_rpc_interface.py
[modify] https://crrev.com/ba232a36e058b58188a662647eeaa52c2b76a945/frontend/afe/rpc_utils.py
[modify] https://crrev.com/ba232a36e058b58188a662647eeaa52c2b76a945/frontend/afe/views.py
[modify] https://crrev.com/ba232a36e058b58188a662647eeaa52c2b76a945/frontend/afe/site_rpc_interface.py
[modify] https://crrev.com/ba232a36e058b58188a662647eeaa52c2b76a945/frontend/afe/site_rpc_interface_unittest.py
[add] https://crrev.com/ba232a36e058b58188a662647eeaa52c2b76a945/frontend/afe/moblab_rpc_interface_unittest.py

Status: Fixed (was: Assigned)

Comment 3 by dchan@chromium.org, Oct 7 2016

Labels: VerifyIn-55

Comment 4 by dchan@chromium.org, Oct 10 2016

Labels: -VerifyIn-55

Comment 5 by dchan@google.com, Nov 19 2016

Labels: VerifyIn-56

Comment 6 by dchan@google.com, Jan 21 2017

Labels: VerifyIn-57

Comment 7 by dchan@google.com, Mar 4 2017

Labels: VerifyIn-58

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

Labels: VerifyIn-59

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

Labels: VerifyIn-60
Labels: VerifyIn-61

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

Status: Archived (was: Fixed)

Sign in to add a comment