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

Issue 792309 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Aug 3
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: ----

Blocked on:
issue 791808
issue 792114

Blocking:
issue 790757



Sign in to add a comment

RPC changes in lab to support skylab migration

Project Member Reported by xixuan@chromium.org, Dec 6 2017

Issue description

Reference doc:
https://docs.google.com/document/d/1YaPZt4T036nlxnoVr0olL_hX-rom9vaANR35TfUoXM4/edit?ts=5a25c8ae

Forward the RPC reading afe_hosts_labels to afe_static_host_labels, reading afe_labels to afe_static_labels.
 
Cc: pprabhu@chromium.org shuqianz@chromium.org
Blocking: 790757
Blockedon: 791808 792114
Project Member

Comment 4 by bugdroid1@chromium.org, Dec 19 2017

Project Member

Comment 5 by bugdroid1@chromium.org, Dec 21 2017

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

commit 9f112777d0b6298288debfa2c443a42f82c9919a
Author: Xixuan Wu <xixuan@chromium.org>
Date: Thu Dec 21 02:32:04 2017

autotest: Add dbclass in afe.models.

BUG= chromium:792309 
TEST=Ran unittest. Ran with local autotest & migrated local mysql.

Change-Id: I993f56f1e9ca355923511657d605cc69bcf3e91b

[modify] https://crrev.com/9f112777d0b6298288debfa2c443a42f82c9919a/frontend/afe/models.py
[modify] https://crrev.com/9f112777d0b6298288debfa2c443a42f82c9919a/scheduler/scheduler_models.py
[modify] https://crrev.com/9f112777d0b6298288debfa2c443a42f82c9919a/frontend/afe/models_test.py

Project Member

Comment 6 by bugdroid1@chromium.org, Dec 21 2017

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

commit e6c686186ecb9992cb69505cb7853c46d12be8df
Author: Xixuan Wu <xixuan@chromium.org>
Date: Thu Dec 21 19:52:16 2017

Revert "autotest: Add dbclass in afe.models."

This reverts commit 9f112777d0b6298288debfa2c443a42f82c9919a.

Reason for revert: <INSERT REASONING HERE>

Original change's description:
> autotest: Add dbclass in afe.models.
> 
> BUG= chromium:792309 
> TEST=Ran unittest. Ran with local autotest & migrated local mysql.
> 
> Change-Id: I993f56f1e9ca355923511657d605cc69bcf3e91b

Bug:  chromium:792309 
Change-Id: I03bff0908c3b67f09b42e3df9616c917891e7ca9
Reviewed-on: https://chromium-review.googlesource.com/840784
Tested-by: Xixuan Wu <xixuan@chromium.org>
Commit-Queue: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Xixuan Wu <xixuan@chromium.org>

[modify] https://crrev.com/e6c686186ecb9992cb69505cb7853c46d12be8df/frontend/afe/models.py
[modify] https://crrev.com/e6c686186ecb9992cb69505cb7853c46d12be8df/scheduler/scheduler_models.py
[modify] https://crrev.com/e6c686186ecb9992cb69505cb7853c46d12be8df/frontend/afe/models_test.py

Project Member

Comment 7 by bugdroid1@chromium.org, Jan 9 2018

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

commit cc12f3e61d8396a66d667eed71a9d0ff3ff28176
Author: Xixuan Wu <xixuan@chromium.org>
Date: Tue Jan 09 09:04:43 2018

autotest: Add db class in afe.models.

BUG= chromium:792309 
TEST=Ran autotest locally with locally migrated db.
Ran unittest.

Change-Id: Ie3f98b5f4473123e7de13de6bf159d51c8772cce
Reviewed-on: https://chromium-review.googlesource.com/848192
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Xixuan Wu <xixuan@chromium.org>

[modify] https://crrev.com/cc12f3e61d8396a66d667eed71a9d0ff3ff28176/frontend/afe/models.py

Project Member

Comment 8 by bugdroid1@chromium.org, Jan 10 2018

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

commit bc3e0c89c3f42c3e59b33857d5ecd5622526460a
Author: Xixuan Wu <xixuan@chromium.org>
Date: Wed Jan 10 23:44:44 2018

autotest: Add endpoints in afe.models.Host to coperate with static
labels.

BUG= chromium:792309 
TEST=1. Ran unittest;
2. Ran with local autotest & migrated local mysql;
3. Ran ./cli/atest host list.

Change-Id: I55bd9d528602b3f300a324dfed9b931fc08f7286
Reviewed-on: https://chromium-review.googlesource.com/833207
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>

[modify] https://crrev.com/bc3e0c89c3f42c3e59b33857d5ecd5622526460a/frontend/afe/models.py
[modify] https://crrev.com/bc3e0c89c3f42c3e59b33857d5ecd5622526460a/frontend/afe/rpc_utils.py
[modify] https://crrev.com/bc3e0c89c3f42c3e59b33857d5ecd5622526460a/frontend/afe/rpc_interface.py

Project Member

Comment 9 by bugdroid1@chromium.org, Jan 11 2018

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

commit 211ae69d00fbcaf6c656735e1ce2464c7e9dc8cc
Author: Xixuan Wu <xixuan@chromium.org>
Date: Thu Jan 11 20:02:12 2018

autotest: Don't delete labels that are replaced by static labels.

BUG= chromium:792309 
TEST=Run './site_utils/label_cleaner.py -p board:' with local migrated
mysql.

Change-Id: Ia263d3b54576b3ed01c9d57b2d1688333ee34875
Reviewed-on: https://chromium-review.googlesource.com/833330
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Xixuan Wu <xixuan@chromium.org>

[modify] https://crrev.com/211ae69d00fbcaf6c656735e1ce2464c7e9dc8cc/site_utils/label_cleaner.py

Project Member

Comment 10 by bugdroid1@chromium.org, Jan 11 2018

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

commit c7049db9da61a54762c5cd63cf97d76a1f027212
Author: Xixuan Wu <xixuan@chromium.org>
Date: Thu Jan 11 20:02:12 2018

autotest: Backup afe db host information with table afe_static_labels.

BUG= chromium:792309 
TEST=Ran backup_mysql_db.py locally with migrated local mysql.

Change-Id: I599efeefcd637cab3009e697b33057d5d6a121d8
Reviewed-on: https://chromium-review.googlesource.com/833489
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Xixuan Wu <xixuan@chromium.org>

[modify] https://crrev.com/c7049db9da61a54762c5cd63cf97d76a1f027212/site_utils/backup_mysql_db.py

Project Member

Comment 12 by bugdroid1@chromium.org, Jan 12 2018

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

commit c820ea892efc9d6fdf60b0ee7bb519ab400ebbb1
Author: Xixuan Wu <xixuan@chromium.org>
Date: Fri Jan 12 22:31:15 2018

autotest: Don't modify static labels in afe_labels.

BUG= chromium:792309 
TEST=1. Ran unittest.
2. Calls RPC locally.

Change-Id: I8f087c596d7b03332155e95c880ac9629c867bcd
Reviewed-on: https://chromium-review.googlesource.com/841143
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>

[modify] https://crrev.com/c820ea892efc9d6fdf60b0ee7bb519ab400ebbb1/frontend/afe/rpc_interface_unittest.py
[modify] https://crrev.com/c820ea892efc9d6fdf60b0ee7bb519ab400ebbb1/frontend/afe/rpc_interface.py

Project Member

Comment 13 by bugdroid1@chromium.org, Jan 12 2018

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

commit 1f4458315360bfbe4251f55de08f91d9074213e3
Author: Xixuan Wu <xixuan@chromium.org>
Date: Fri Jan 12 22:31:08 2018

autotest: Remove dead codes in query_managers.

BUG= chromium:792309 
TEST=Ran unittest.
Ran autotest locally.

Change-Id: I806a00ea51e7ae5d6aa9362b0747390d6f440709
Reviewed-on: https://chromium-review.googlesource.com/849637
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Xixuan Wu <xixuan@chromium.org>

[modify] https://crrev.com/1f4458315360bfbe4251f55de08f91d9074213e3/scheduler/query_managers.py

Project Member

Comment 14 by bugdroid1@chromium.org, Jan 18 2018

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

commit e7706031001503f1840d0b62b227c42bf59df102
Author: Xixuan Wu <xixuan@chromium.org>
Date: Thu Jan 18 22:27:51 2018

autotest: Modify RPCs about add labels to hosts.

BUG= chromium:792309 
TEST=1. Ran unittest.
2. Ran RPC 'label_add_hosts' and 'host_add_labels' locally.
3. Ran ./cli/atest label add <label_name> -m <host_name>

Change-Id: I5329ec381c3b466cc6ad8687c8e5fa15cf91d525
Reviewed-on: https://chromium-review.googlesource.com/841531
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>

[modify] https://crrev.com/e7706031001503f1840d0b62b227c42bf59df102/frontend/afe/models.py
[modify] https://crrev.com/e7706031001503f1840d0b62b227c42bf59df102/frontend/afe/rpc_interface_unittest.py
[modify] https://crrev.com/e7706031001503f1840d0b62b227c42bf59df102/frontend/afe/rpc_interface.py

Project Member

Comment 15 by bugdroid1@chromium.org, Jan 19 2018

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

commit 8c7113a941511fcf77a05602142e7525dea7bae3
Author: Xixuan Wu <xixuan@chromium.org>
Date: Fri Jan 19 00:54:41 2018

autotest: Use Host.get_hosts_with_labels to find hosts in scheduler.

BUG= chromium:792309 
TEST=Ran scheduling suites with local autotest.
Ran unittest.

Change-Id: I444f703dd4a9114073d7908999d21344ed590032
Reviewed-on: https://chromium-review.googlesource.com/858487
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Xixuan Wu <xixuan@chromium.org>

[modify] https://crrev.com/8c7113a941511fcf77a05602142e7525dea7bae3/frontend/afe/models.py
[modify] https://crrev.com/8c7113a941511fcf77a05602142e7525dea7bae3/frontend/afe/rpc_utils.py
[modify] https://crrev.com/8c7113a941511fcf77a05602142e7525dea7bae3/scheduler/rdb.py

Project Member

Comment 16 by bugdroid1@chromium.org, Jan 22 2018

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

commit aae18b4b4d8ae1c5ad9371d9a3090ca64142105c
Author: Xixuan Wu <xixuan@chromium.org>
Date: Mon Jan 22 20:43:43 2018

autotest: change scheduler_model.py to adapt to static labels.

BUG= chromium:792309 
TEST=Ran unittest.

Change-Id: Ia6e7c6aff502f047aa2081db3db6673c5c1d6580
Reviewed-on: https://chromium-review.googlesource.com/861961
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>

[modify] https://crrev.com/aae18b4b4d8ae1c5ad9371d9a3090ca64142105c/scheduler/scheduler_models_unittest.py
[modify] https://crrev.com/aae18b4b4d8ae1c5ad9371d9a3090ca64142105c/scheduler/scheduler_models.py

Project Member

Comment 17 by bugdroid1@chromium.org, Jan 25 2018

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

commit 79cbf49949a471baeffb67f47e745c12396ff7fb
Author: Xixuan Wu <xixuan@chromium.org>
Date: Thu Jan 25 04:27:09 2018

autotest: Make 'get_labels' respect static labels.

BUG= chromium:792309 
TEST=Ran unittest.
Ran RPC get_labels directly with local mysql.

Change-Id: I0b0763fe0d6f9c9b3eef41b12c8a3422897e2f3c
Reviewed-on: https://chromium-review.googlesource.com/862843
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>

[modify] https://crrev.com/79cbf49949a471baeffb67f47e745c12396ff7fb/frontend/afe/rpc_interface_unittest.py
[modify] https://crrev.com/79cbf49949a471baeffb67f47e745c12396ff7fb/frontend/afe/rpc_interface.py

Project Member

Comment 18 by bugdroid1@chromium.org, Jan 26 2018

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

commit 040fc8b28c188e8f909a837f8317f10077f05ba9
Author: Xixuan Wu <xixuan@chromium.org>
Date: Fri Jan 26 17:16:43 2018

autotest: Modify RPC 'label_remove_hosts' & 'host_remove_labels'

BUG= chromium:792309 
TEST=Ran unittest.
Call RPC locally with local Mysql.

Change-Id: I73a054f104c136293d57f6bba1659b0f0d700382
Reviewed-on: https://chromium-review.googlesource.com/865555
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Shuqian Zhao <shuqianz@chromium.org>

[modify] https://crrev.com/040fc8b28c188e8f909a837f8317f10077f05ba9/frontend/afe/models.py
[modify] https://crrev.com/040fc8b28c188e8f909a837f8317f10077f05ba9/frontend/afe/rpc_interface_unittest.py
[modify] https://crrev.com/040fc8b28c188e8f909a837f8317f10077f05ba9/frontend/afe/rpc_interface.py

Project Member

Comment 19 by bugdroid1@chromium.org, Jan 26 2018

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

commit 20b323f23f56e15e506d3849f48e37ee3b86f4b8
Author: Xixuan Wu <xixuan@chromium.org>
Date: Fri Jan 26 22:09:48 2018

autotest: Modify RPC remove_board_from_shard

BUG= chromium:792309 
TEST=Ran unittest.

Change-Id: I0b1f37648501ae06d39818c79646be4cebd6e699
Reviewed-on: https://chromium-review.googlesource.com/865638
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Shuqian Zhao <shuqianz@chromium.org>

[modify] https://crrev.com/20b323f23f56e15e506d3849f48e37ee3b86f4b8/frontend/afe/rpc_interface_unittest.py
[modify] https://crrev.com/20b323f23f56e15e506d3849f48e37ee3b86f4b8/frontend/afe/rpc_interface.py

Project Member

Comment 20 by bugdroid1@chromium.org, Jan 27 2018

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

commit 2dc10cea03bf146ce5a516a0d5ffc1e6de470c46
Author: Xixuan Wu <xixuan@chromium.org>
Date: Sat Jan 27 01:00:55 2018

autotest: Make job dependencies respect RESPECT_STATIC_LABEL

BUG= chromium:792309 
TEST=Ran unittest.

Change-Id: I22356aee23892661d382dc3473ac685fc90d72f7
Reviewed-on: https://chromium-review.googlesource.com/869131
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Shuqian Zhao <shuqianz@chromium.org>

[modify] https://crrev.com/2dc10cea03bf146ce5a516a0d5ffc1e6de470c46/frontend/afe/rpc_interface_unittest.py
[modify] https://crrev.com/2dc10cea03bf146ce5a516a0d5ffc1e6de470c46/frontend/afe/rpc_utils.py

Project Member

Comment 21 by bugdroid1@chromium.org, Jan 27 2018

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

commit ab2b93671ed54c266768b4b95fce8907e7c68b0c
Author: Xixuan Wu <xixuan@chromium.org>
Date: Sat Jan 27 01:00:56 2018

autotest: Let moblab_interface respect static labels.

BUG= chromium:792309 
TEST=Ran unittest.

Change-Id: I7ae072a44bf31ae6bfe4be1d429bfcc1d99e9ab2
Reviewed-on: https://chromium-review.googlesource.com/869141
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Shuqian Zhao <shuqianz@chromium.org>

[modify] https://crrev.com/ab2b93671ed54c266768b4b95fce8907e7c68b0c/frontend/afe/moblab_rpc_interface.py

Project Member

Comment 22 by bugdroid1@chromium.org, Jan 29 2018

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

commit 4379e9309115337fd49d58140ad1a70c9790e69f
Author: Xixuan Wu <xixuan@chromium.org>
Date: Mon Jan 29 01:11:59 2018

autotest: Make 'assign_to_shard' respect static labels.

BUG= chromium:792309 
TEST=Ran unittest.

Change-Id: I681b9f947ab7e80d1d307c679dc0a5aa8911fb6b
Reviewed-on: https://chromium-review.googlesource.com/871047
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Xixuan Wu <xixuan@chromium.org>

[modify] https://crrev.com/4379e9309115337fd49d58140ad1a70c9790e69f/frontend/afe/models.py
[modify] https://crrev.com/4379e9309115337fd49d58140ad1a70c9790e69f/frontend/afe/rpc_interface_unittest.py

Project Member

Comment 23 by bugdroid1@chromium.org, Feb 15 2018

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

commit b16b58f261af2e017964f3b4b756daa87232ec8c
Author: Xixuan Wu <xixuan@chromium.org>
Date: Thu Feb 15 22:38:58 2018

autotest: add new Table afe_static_host_attributes

BUG= chromium:792309 
TEST=Ran 'migrate.py sync 127' locally.

Change-Id: I6fbd9073c5205d8b178f28065d8041ae06f3c222
Reviewed-on: https://chromium-review.googlesource.com/922450
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Shuqian Zhao <shuqianz@chromium.org>

[add] https://crrev.com/b16b58f261af2e017964f3b4b756daa87232ec8c/frontend/migrations/127_add_afe_static_host_attributes.py

Project Member

Comment 24 by bugdroid1@chromium.org, Feb 23 2018

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

commit e60b4013cc3ed0b73e853ba26f227bb8d5da09cd
Author: Xixuan Wu <xixuan@chromium.org>
Date: Fri Feb 23 23:12:52 2018

Project Member

Comment 25 by bugdroid1@chromium.org, Feb 23 2018

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

commit 903087bcc53368167631aa3c84b96c8bf648cca0
Author: Xixuan Wu <xixuan@chromium.org>
Date: Fri Feb 23 23:38:33 2018

autotest: Add django model class for table afe_static_host_attributes.

BUG= chromium:792309 
TEST=Ran 'dummy_pass' in local autotest.

Change-Id: Ib289113990ef80d473a5c74874e2f11c77b9429a
Reviewed-on: https://chromium-review.googlesource.com/927747
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Shuqian Zhao <shuqianz@chromium.org>

[modify] https://crrev.com/903087bcc53368167631aa3c84b96c8bf648cca0/frontend/afe/models.py

Project Member

Comment 26 by bugdroid1@chromium.org, Feb 24 2018

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

commit 66c222618d88ce0c2d87bffd44b2415d1c795f5b
Author: Xixuan Wu <xixuan@chromium.org>
Date: Sat Feb 24 05:07:10 2018

autotest: ensure we can fall back to use skylab configs in global_config.

BUG= chromium:792309 
TEST=None

Change-Id: I74b7d311c2d6beb2a356a8b76524b73771340c25
Reviewed-on: https://chromium-review.googlesource.com/935676
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Shuqian Zhao <shuqianz@chromium.org>

[modify] https://crrev.com/66c222618d88ce0c2d87bffd44b2415d1c795f5b/global_config.ini

Project Member

Comment 27 by bugdroid1@chromium.org, Feb 28 2018

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

commit bd9d5e4759d00aca07a807864bf8bba29aed37e4
Author: Xixuan Wu <xixuan@chromium.org>
Date: Wed Feb 28 01:25:59 2018

autotest: Make 'get_hosts' respect static attributes & labels.

BUG= chromium:792309 
TEST=Ran unittest. Call RPC 'get_hosts' locally.

Change-Id: I1fd5cbe59781fd3c17816c5e386cffc05635c0ff
Reviewed-on: https://chromium-review.googlesource.com/929609
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Shuqian Zhao <shuqianz@chromium.org>

[modify] https://crrev.com/bd9d5e4759d00aca07a807864bf8bba29aed37e4/frontend/afe/rpc_utils.py
[modify] https://crrev.com/bd9d5e4759d00aca07a807864bf8bba29aed37e4/frontend/afe/rpc_interface_unittest.py
[modify] https://crrev.com/bd9d5e4759d00aca07a807864bf8bba29aed37e4/frontend/afe/rpc_interface.py

Project Member

Comment 28 by bugdroid1@chromium.org, Mar 1 2018

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

commit 91ecef5829e0587b802247d12f2b266408c7c3ae
Author: Xixuan Wu <xixuan@chromium.org>
Date: Thu Mar 01 04:14:34 2018

autotest: Make RPC 'get_hosts_by_attribute' respect static attributes.

BUG= chromium:792309 
TEST=Ran unittest. Ran 'dummy_Pass' with local autotest.
Call RPC 'get_hosts_by_attribute' locally.

Change-Id: I61ad23fdb124300d93d70fe3d1dafc757fe03298
Reviewed-on: https://chromium-review.googlesource.com/930177
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Shuqian Zhao <shuqianz@chromium.org>

[modify] https://crrev.com/91ecef5829e0587b802247d12f2b266408c7c3ae/frontend/afe/rpc_interface_unittest.py
[modify] https://crrev.com/91ecef5829e0587b802247d12f2b266408c7c3ae/frontend/afe/rpc_interface.py

Project Member

Comment 29 by bugdroid1@chromium.org, Mar 1 2018

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

commit 2975f40890696fbf63ba9cf1d93f6fdd40a10953
Author: Xixuan Wu <xixuan@chromium.org>
Date: Thu Mar 01 20:24:08 2018

autotest: Make RPC 'get_host_attributes' respect static attributes

BUG= chromium:792309 
TEST=Ran unittest. Call RPC 'get_host_attributes' locally.

Change-Id: Iae7dccfc0c63481f07c6b4122774e7f5e02f2bd0
Reviewed-on: https://chromium-review.googlesource.com/929889
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Shuqian Zhao <shuqianz@chromium.org>

[modify] https://crrev.com/2975f40890696fbf63ba9cf1d93f6fdd40a10953/frontend/afe/rpc_interface_unittest.py
[modify] https://crrev.com/2975f40890696fbf63ba9cf1d93f6fdd40a10953/frontend/afe/rpc_interface.py

Status: Verified (was: Assigned)

Sign in to add a comment