Issue metadata
Sign in to add a comment
|
Catapult roll failing telemetry_perf_unittests - process_perf_results_unittest.ProcessPerfResultsIntegrationTest.testIntegration |
||||||||||||||||||||||
Issue descriptionLooks like the rolls have been failing for more than a day. At least all the rolls present in the auto-roller dashboard: https://autoroll.skia.org/r/catapult-autoroll The test is failing on linux_chromium_rel_ng, android-marshmallow-arm64-rel, and possibly other configs. Example builds: https://ci.chromium.org/p/chromium/builders/luci.chromium.try/android-marshmallow-arm64-rel/167532 https://ci.chromium.org/p/chromium/builders/luci.chromium.try/linux_chromium_rel_ng/271715 https://ci.chromium.org/p/chromium/builders/luci.chromium.try/linux_chromium_rel_ng/271790 Example CL: https://chromium-review.googlesource.com/c/chromium/src/+/1409094 The failing test is always "process_perf_results_unittest.ProcessPerfResultsIntegrationTest.testIntegration" in "telemetry_perf_unittests". Brian can you help triage this? I don't know who the right sheriff is for capapult, is that the perf sheriff? This is blocking fixing gtest test scaling.
,
Jan 15
Ben, can you triage? It looks like it's been failing since this roll: https://chromium-review.googlesource.com/c/chromium/src/+/1407872 Which has 2 of your CLs: 2019-01-11 benjhayden@chromium.org Remove TagMap. 2019-01-11 benjhayden@chromium.org Truncate serialized floats in HistogramSet JSON.
,
Jan 15
Remove TagMap was reverted. Now trying to figure out how to repro before relanding.
,
Jan 15
Command to run is: ./tools/perf/run_tests --test-filter=process_perf_results_unittest.ProcessPerfResultsIntegrationTest.testIntegration I think
,
Jan 15
perfbot-sheriff would be the right person, but there's no perfbot sheriff right now :(. Working on it.
,
Jan 15
The new roll is passing: https://chromium-review.googlesource.com/c/chromium/src/+/1413252 Thanks Juan for the revert!
,
Jan 15
Yes, thank you for unblocking the roller.
,
Jan 16
(6 days ago)
,
Jan 16
(6 days ago)
,
Jan 17
(5 days ago)
No problem. Also, sorry for submitting one of the CLs that contributed to the breakage. 😅 |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by jmadill@google.com
, Jan 15