commit | bf3ea1f74957c31b39f16186c841c9f9a84d2235 | [log] [tgz] |
---|---|---|
author | Scott Lee <ddoman@chromium.org> | Fri Sep 25 19:56:28 2020 |
committer | LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> | Fri Sep 25 19:56:28 2020 |
tree | 5a0267f35135804b79510f3f08e6bfaea0a49d98 | |
parent | f946f76947daa884a89e3c6ea922be6cedb8bf0f [diff] |
[resultdb] set test_location_base if resultdb config has it Chromium-Deriver sets the test-location-base of the tests with the values at https://source.chromium.org/chromium/infra/infra/+/master:go/src/go.chromium.org/luci/resultdb/internal/services/deriver/swarming.go;l=188-194 The current plan is to move the test location base config to test_suites.pyl. In this CL, maybe_enable_resultdb_for_test() looks up the key from resultdb config, and pass the value to api.resultdb.wrap(). R=chanli@chromium.org,gbeaty@chromium.org,martiniss@chromium.org,nodir@chromium.org CC=iannucci@chromium.org Bug: 1099573 Change-Id: I39a6cd73e8d8b8c9d512c5441f4ffd2328e4747b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/build/+/2419502 Commit-Queue: Scott Lee <ddoman@chromium.org> Reviewed-by: Dirk Pranke <dpranke@google.com> Reviewed-by: Chan Li <chanli@chromium.org> Reviewed-by: Nodir Turakulov <nodir@chromium.org>
If you're here to make a change to ‘recipes’ (the code located in scripts/slave/recipes*
), please take a look at the README for more information pertaining to recipes.
The preferred style is PEP8 with two-space indent; that is, the Chromium Python style. Functions use lowercase_with_underscores
, with the exception of the special functions RunSteps
and RunTests
in recipes. Use yapf (git cl format --no-clang-format
) to autoformat new code.