commit | bc42ddc035859d9c1b316ee7d20de0b4da54d01f | [log] [tgz] |
---|---|---|
author | Dean Michael Berris <dberris@google.com> | Thu Nov 26 03:11:49 2020 |
committer | LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> | Thu Nov 26 03:11:49 2020 |
tree | 62df58405d293803808c2603bb0bde9823b57f46 | |
parent | 987acccd410ebbea7b321f37a55713665c570166 [diff] |
chromeperf: Handle responses from legacy service This change makes the server handle the HTTP result codes from the legacy service, and propagates the issue to the client. This code is only useful in the interim because we're proxying requests to the legacy service. In the future these error messages will not make it to the client, as we'll have a better logging and debugging strategy in production for this gRPC service. R=abennetts@google.com, fancl@chromium.org Bug: chromium:1059667 Change-Id: I2b245420687538bdc3418c7244dba1224603ed99 Reviewed-on: https://chromium-review.googlesource.com/c/infra/infra/+/2560425 Commit-Queue: Dean Berris <dberris@chromium.org> Commit-Queue: Andrew Bennetts <abennetts@google.com> Auto-Submit: Dean Berris <dberris@chromium.org> Reviewed-by: Andrew Bennetts <abennetts@google.com> Cr-Commit-Position: refs/heads/master@{#36735}
Testing Welcome to the Chrome Infra repository!
Wondering where to start? Check out General Chrome Infrastructure documentation. In particular, to check out this repo and the rest of the infrastructure code, follow the instructions here. The rest of this page is specific to this repo.
sys.path
modifications.from infra.path_hacks.common import <stg>
is actually getting <stg>
from build/scripts/common.git push origin <updated hash>:deployed
git push
commandIf you've added a new module, run your tests with test.py:
*_test.py
files to this directory.Double-check that your tests are getting picked up when you want them to be: ./test.py test <path-to-package>
.
Tests still not getting picked up by test.py? Double-check to make sure you have __init__.py
files in each directory of your module so Python recognizes it as a package.
The preferred style is PEP8 with two-space indent; that is, the Chromium Python style, except functions use lowercase_with_underscores
. Use yapf (git cl format
) to autoformat new code.