| 5ae0a003b1c1ab6e2a9cd73f7899fb1f22c8f14b |
|
05-May-2017 |
Aviv Keshet <akeshet@chromium.org> |
autotest: clean up various uses of logging.exception
Many existing callers use logging.exception incorrectly, resulting in
exceptions being printed twice. Fix a bunch of them in server/
directory.
BUG=None
TEST=None
Change-Id: I03fc7ece34abbeca8bd8c328ac7cbb33c84bdc33
Reviewed-on: https://chromium-review.googlesource.com/497611
Commit-Ready: Aviv Keshet <akeshet@chromium.org>
Tested-by: Aviv Keshet <akeshet@chromium.org>
Reviewed-by: Allen Li <ayatane@chromium.org>
/external/autotest/server/hosts/base_label.py
|
| b6cea61e59b40819af04fbf9ced7dba75785d43e |
|
10-Feb-2017 |
Prathmesh Prabhu <pprabhu@chromium.org> |
Replaces usage of afe_utils.get_labels with HostInfo properties
BUG=chromium:678430
TEST=None
Change-Id: I3502e1d6baddd9f598634077854030005ceef290
Reviewed-on: https://chromium-review.googlesource.com/440563
Commit-Ready: Prathmesh Prabhu <pprabhu@chromium.org>
Tested-by: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Allen Li <ayatane@chromium.org>
/external/autotest/server/hosts/base_label.py
|
| 1b2704cdad9af67cc0d38859ca07805be885e418 |
|
10-Jan-2017 |
Prathmesh Prabhu <pprabhu@chromium.org> |
Add AfeStore implementation of CachingHostInfoStore.
This is a CachingHostInfoStore that uses AFE RPC calls to refresh /
update host information from the infrastructure.
The implementation currently doesn't update attributes. We'll fill that
in when we replace the first attribute update call to go via HostInfo.
BUG=chromium:678430
TEST=new unittests.
Change-Id: I40e77f0005af44a142845c79d02f75b10e638b29
Reviewed-on: https://chromium-review.googlesource.com/426012
Commit-Ready: Prathmesh Prabhu <pprabhu@chromium.org>
Tested-by: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Allen Li <ayatane@chromium.org>
/external/autotest/server/hosts/base_label.py
|
| eee38e0baff993cf20229ea825b514e7f806cb79 |
|
31-Aug-2016 |
Kevin Cheng <kevcheng@chromium.org> |
[autotest] Add decorator for labels to stay forever.
BUG=chromium:621640
TEST=locally verified label still gets checked and if it already exists,
will return True.
Change-Id: I09d73ea06db8da170e03ec919f2db2bb80d7ee43
Reviewed-on: https://chromium-review.googlesource.com/378555
Reviewed-by: Kevin Cheng <kevcheng@chromium.org>
Commit-Queue: Kevin Cheng <kevcheng@chromium.org>
Tested-by: Kevin Cheng <kevcheng@chromium.org>
/external/autotest/server/hosts/base_label.py
|
| 05ae2a45397188dd7d7adbc49a61095ebf85e151 |
|
06-Jun-2016 |
Kevin Cheng <kevcheng@chromium.org> |
[autotest] Store afe_host in and remove host attributes to/from machine dict.
There are multiple times when the AFE is queried for random bits of info
(host attibutes, labels, etc). To reduce the number of calls, store the
AFE host object into the machine dict so we can store it into the host class.
I chose the AbstractSSHHost class to grab it out of the arg dict since that
is used both by CrosHost and ADBHost.
Also removed host attributes as a class instance attribute since
afe_host contains all that info already.
Also use the os labels to do host class detection.
BUG=chromium:546741
TEST=Tested a couple scenarios:
- Manually on moblab to make sure cros/adb/testbed devices all pass
dummy tests.
- Trybot job on gnawty-paladin.
- Moblab dummy suite run to ensure no breaking backward compatibility changes
(moblab on R52 and testing with a tryjob gnawty build).
- test_{that,droid}.py on a cros/adbhost device.
Change-Id: I101c478c0f27f9fd444ccd4699a5c94e76d3b17e
Reviewed-on: https://chromium-review.googlesource.com/349990
Commit-Ready: Kevin Cheng <kevcheng@chromium.org>
Tested-by: Kevin Cheng <kevcheng@chromium.org>
Reviewed-by: Kevin Cheng <kevcheng@chromium.org>
/external/autotest/server/hosts/base_label.py
|
| 87166aad156735a02fe732c7dbe2f836fb48b6a7 |
|
07-Jun-2016 |
Kevin Cheng <kevcheng@chromium.org> |
[autotest] Add in additional logging when updating labels.
BUG=None
TEST=None
Change-Id: I73fd789314c735d7e5ec23425656038de27ebe17
Reviewed-on: https://chromium-review.googlesource.com/350410
Commit-Ready: Kevin Cheng <kevcheng@chromium.org>
Tested-by: Kevin Cheng <kevcheng@chromium.org>
Reviewed-by: Richard Barnette <jrbarnette@google.com>
/external/autotest/server/hosts/base_label.py
|
| a04ec4b85ad6231cf157fa94d1e4427ac669243d |
|
04-May-2016 |
Kevin Cheng <kevcheng@chromium.org> |
[autotest] Add extra logging when adding/removing labels.
BUG=None
TEST=None
Change-Id: Iaa318467109a9ace7d47f395bac95bb85584ea7f
Reviewed-on: https://chromium-review.googlesource.com/342369
Commit-Ready: Kevin Cheng <kevcheng@chromium.org>
Tested-by: Kevin Cheng <kevcheng@chromium.org>
Reviewed-by: Richard Barnette <jrbarnette@chromium.org>
/external/autotest/server/hosts/base_label.py
|
| bdfc57d71f70af729d94e547752aec1f0c41a8fd |
|
14-Apr-2016 |
Kevin Cheng <kevcheng@chromium.org> |
[autotest] Update LabelRetriever to create labels everywhere.
This change does one major thing:
1. Update the rpc_interface host_add_labels() to create labels
if they don't exist on the master first, then to the shards.
With these changes, we should be creating labels on the master AFE
and propagating those changes to the shards properly.
BUG=chromium:603420
TEST=Locally on moblab and on test server with shard, verified label
gets created on test server and shard with same label ids. Also all
unittests pass.
Change-Id: I3fd88ae3cca32702f8b537f673d6db2941e3e7f7
Reviewed-on: https://chromium-review.googlesource.com/338904
Commit-Ready: Kevin Cheng <kevcheng@chromium.org>
Tested-by: Kevin Cheng <kevcheng@chromium.org>
Reviewed-by: Dan Shi <dshi@google.com>
/external/autotest/server/hosts/base_label.py
|
| c48cf599a89faf6ae04b5b5f7fb2fde9a61fc60e |
|
11-Apr-2016 |
Kevin Cheng <kevcheng@chromium.org> |
[autotest] Modify BaseLabel to populate labels only when updating them.
Moving the _populate_known_labels method to right before updating them
since that's the only time we need it and it reduces the initialization
load for CrosHost.
BUG=None
TEST=None
Change-Id: Ib3199381c9338016ef55d60f5f4c64d9d3af08a2
Reviewed-on: https://chromium-review.googlesource.com/338105
Commit-Ready: Kevin Cheng <kevcheng@chromium.org>
Tested-by: Kevin Cheng <kevcheng@chromium.org>
Reviewed-by: Richard Barnette <jrbarnette@chromium.org>
/external/autotest/server/hosts/base_label.py
|
| 61b1282b3b49922f50b3e13b3d62411ac3145900 |
|
06-Apr-2016 |
Kevin Cheng <kevcheng@chromium.org> |
[autotest] Create labels before adding them to a host.
Some HWID generated labels might not be in the label db when we add it
to a host so create them first.
BUG=chromium:571087
TEST=locally test on moblab that labels are properly created only when
needed.
Change-Id: Ib7a729cf9d506a9667cbfd5169bdfaf006eaaf82
Reviewed-on: https://chromium-review.googlesource.com/337355
Commit-Ready: Kevin Cheng <kevcheng@chromium.org>
Tested-by: Kevin Cheng <kevcheng@chromium.org>
Reviewed-by: Richard Barnette <jrbarnette@chromium.org>
/external/autotest/server/hosts/base_label.py
|
| c6a645adaf6f134d9457cf8b5d3408d66bfbb1ef |
|
18-Dec-2015 |
Kevin Cheng <kevcheng@chromium.org> |
[autotest] Create new Base Label class.
This is to create the scaffolding for the base label scheme and to move
ADBHost and TestBed over since they only have one label for now. A
separate cl will move CrosHost over since there are many more labels
there.
Also add in the os label for ADBHost.
BUG=chromium:571085
TEST=Ensure adbhost and testbed create the same labels as before in
addition to the os label and unittests
Change-Id: Ief3961d05be030f81fe7cb8c9846f04c4e92b8e5
Reviewed-on: https://chromium-review.googlesource.com/319483
Commit-Ready: Kevin Cheng <kevcheng@chromium.org>
Tested-by: Kevin Cheng <kevcheng@chromium.org>
Reviewed-by: Kevin Cheng <kevcheng@chromium.org>
/external/autotest/server/hosts/base_label.py
|