Add commit positions to BQ test results for chromium/src |
|||
Issue descriptionCurrently test results in BQ do not include commit positions. Findit would need such data to measure flakiness of a single test.
,
Jan 25 2018
,
Jan 25 2018
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/363def7855aad680839176800a05654945bdb149 commit 363def7855aad680839176800a05654945bdb149 Author: Sean McCullough <seanmccullough@chromium.org> Date: Thu Jan 25 21:53:04 2018 [test-results] add chromium_revision to bigquery table. Bug: 804973 Change-Id: Ia8f36ba989d8db3090e988ab3f74079d5656a9e0 Reviewed-on: https://chromium-review.googlesource.com/887354 Reviewed-by: Tiffany Zhang <zhangtiff@chromium.org> Reviewed-by: Shuotao Gao <stgao@chromium.org> Commit-Queue: Sean McCullough <seanmccullough@chromium.org> [modify] https://crrev.com/363def7855aad680839176800a05654945bdb149/go/src/infra/appengine/test-results/model/gen/testresults.pb.go [modify] https://crrev.com/363def7855aad680839176800a05654945bdb149/go/src/infra/appengine/test-results/frontend/event_logging.go [modify] https://crrev.com/363def7855aad680839176800a05654945bdb149/go/src/infra/appengine/test-results/model/gen/testresults.proto
,
Jan 25 2018
This is live: https://bigquery.cloud.google.com/results/test-results-hrd:bquijob_5ba6511f_1612f606518
,
Jan 26 2018
Many thanks!
nit: some revision is a string like "refs/heads/master@{#21769}".
,
Jan 26 2018
Yeah I saw that. There's no enforcement of consistency in the upload scripts, some of which don't even set a value for that field (it's optional per the spec). They can put whatever they like in it :/ |
|||
►
Sign in to add a comment |
|||
Comment 1 by st...@chromium.org
, Jan 24 2018