diff options
| author | Don Garrett <dgarrett@google.com> | 2015-05-21 19:24:28 -0700 |
|---|---|---|
| committer | ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> | 2015-05-28 00:51:51 +0000 |
| commit | f2904ae8cee464b319fe32d1504634a60ac95392 (patch) | |
| tree | 91f4b8e58527fa76c425f05ba910ab18c78163fa /cbuildbot/binhost.py | |
| parent | f5909602f1602aad1a3683681e4e9b48f068dc52 (diff) | |
| download | chromite-f2904ae8cee464b319fe32d1504634a60ac95392.tar.gz | |
config_lib: Move GetSlavesForMaster.
Move this function from generate_chromeos_config to config_lib.Config,
and update all call sites.
The unittests related to this method seem to really be testing the
configuration information, not the method itself, so they remain in
generate_chromeos_config_unittest.
BUG=chromium:483566
TEST=cros lint + run_tests
Change-Id: Ia9b38b4c868c6800df77c819dcb7ba63448f7893
Reviewed-on: https://chromium-review.googlesource.com/272831
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Don Garrett <dgarrett@chromium.org>
Commit-Queue: Don Garrett <dgarrett@chromium.org>
Tested-by: Don Garrett <dgarrett@chromium.org>
Diffstat (limited to 'cbuildbot/binhost.py')
| -rw-r--r-- | cbuildbot/binhost.py | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/cbuildbot/binhost.py b/cbuildbot/binhost.py index fc1e2b743..6f45ecf97 100644 --- a/cbuildbot/binhost.py +++ b/cbuildbot/binhost.py @@ -64,9 +64,11 @@ def GetChromePrebuiltConfigs(): Returns: A dict mapping BoardKey objects to configs. """ + all_configs = cbuildbot_config.GetConfig() + boards = {} - master_chromium_pfq = cbuildbot_config.GetConfig()['master-chromium-pfq'] - for config in cbuildbot_config.GetSlavesForMaster(master_chromium_pfq): + master_chromium_pfq = all_configs['master-chromium-pfq'] + for config in all_configs.GetSlavesForMaster(master_chromium_pfq): if config.prebuilts: for board in config.boards: boards[GetBoardKey(config, board)] = config |