diff options
author | Jian Cai <jiancai@google.com> | 2021-04-22 11:22:59 -0700 |
---|---|---|
committer | Jian Cai <jiancai@google.com> | 2021-04-28 21:08:09 +0000 |
commit | 2a0c294fd27c0cc02d4c1d9b0e60c844411e7144 (patch) | |
tree | 07356f37445f16354473f16c42bac2ec37562afe | |
parent | 4a12a1285b9992b158afbbc79c91244da4b7cc4b (diff) | |
download | toolchain-utils-2a0c294fd27c0cc02d4c1d9b0e60c844411e7144.tar.gz |
default_remotes: update the list of machines
Synchronize the list of the lab machines allocated by the hardware lab
to the toolchain pool. Also add a description of this file to README.
BUG=chromium:1201755
TEST=None
Change-Id: Ie10d0a881b34dfd0d1b49e6892a4fdf807830c95
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2847103
Reviewed-by: Caroline Tice <cmtice@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Jian Cai <jiancai@google.com>
-rw-r--r-- | crosperf/README.md (renamed from crosperf/experiment_files/README.md) | 8 | ||||
-rw-r--r-- | crosperf/default_remotes | 12 | ||||
-rwxr-xr-x | crosperf/experiment_factory_unittest.py | 10 |
3 files changed, 16 insertions, 14 deletions
diff --git a/crosperf/experiment_files/README.md b/crosperf/README.md index 8c1fe200..18601b67 100644 --- a/crosperf/experiment_files/README.md +++ b/crosperf/README.md @@ -1,4 +1,4 @@ -# Experiment files +# experiment_files To use these experiment files, replace the board, remote and images placeholders and run crosperf on them. @@ -37,3 +37,9 @@ benchmark: page_cycler_v2.morejs { iterations: 1 } ``` + +# default_remotes + +This is the list of machines allocated for toolchain team. +This should be kept in sync with: +https://chromeos-swarming.appspot.com/botlist?c=id&c=task&c=label-board&c=label-pool&c=os&c=status&d=asc&f=label-pool%3Atoolchain&k=label-pool&s=id diff --git a/crosperf/default_remotes b/crosperf/default_remotes index 4aa2a1f6..ac4c3856 100644 --- a/crosperf/default_remotes +++ b/crosperf/default_remotes @@ -1,9 +1,9 @@ -bob : chromeos2-row10-rack9-host1.cros chromeos2-row10-rack9-host3.cros -coral : chromeos2-row9-rack9-host9.cros chromeos2-row9-rack9-host11.cros chromeos2-row9-rack9-host13.cros -elm : chromeos2-row10-rack9-host19.cros chromeos2-row10-rack9-host21.cros +bob : chromeos2-row10-rack9-host1.cros chromeos2-row10-rack9-host3.cros chell : chromeos2-row9-rack9-host1.cros chromeos2-row9-rack9-host3.cros +coral : chromeos2-row9-rack9-host9.cros chromeos2-row9-rack9-host11.cros chromeos2-row9-rack9-host13.cros +elm : chromeos2-row10-rack9-host21.cros kefka : chromeos6-row6-rack22-host2.cros chromeos6-row6-rack22-host3.cros -lulu : chromeos2-row9-rack9-host5.cros chromeos2-row9-rack9-host7.cros +lulu : chromeos2-row9-rack9-host5.cros chromeos2-row9-rack9-host7.cros nautilus : chromeos2-row10-rack9-host9.cros chromeos2-row10-rack9-host11.cros -snappy : chromeos2-row10-rack9-host5.cros chromeos2-row10-rack9-host7.cros -veyron_tiger : chromeos2-row9-rack9-host17.cros chromeos2-row9-rack9-host19.cros +snappy : chromeos2-row10-rack9-host5.cros chromeos2-row10-rack9-host7.cros +veyron_tiger : chromeos2-row9-rack9-host17.cros diff --git a/crosperf/experiment_factory_unittest.py b/crosperf/experiment_factory_unittest.py index 53d28f74..78cf780c 100755 --- a/crosperf/experiment_factory_unittest.py +++ b/crosperf/experiment_factory_unittest.py @@ -399,21 +399,17 @@ class ExperimentFactoryTest(unittest.TestCase): def test_get_default_remotes(self): board_list = [ - 'elm', 'bob', 'chell', 'kefka', 'lulu', 'nautilus', 'snappy', + 'bob', 'chell', 'coral', 'elm', 'kefka', 'lulu', 'nautilus', 'snappy', 'veyron_tiger' ] ef = ExperimentFactory() self.assertRaises(Exception, ef.GetDefaultRemotes, 'bad-board') - # Verify that we have entries for every board, and that we get at least - # two machines for each board. + # Verify that we have entries for every board for b in board_list: remotes = ef.GetDefaultRemotes(b) - if b == 'daisy': - self.assertEqual(len(remotes), 1) - else: - self.assertGreaterEqual(len(remotes), 2) + self.assertGreaterEqual(len(remotes), 1) @mock.patch.object(command_executer.CommandExecuter, 'RunCommand') @mock.patch.object(os.path, 'exists') |