catapult-project / catapult

Deprecated Catapult GitHub. Please instead use http://crbug.com "Speed>Benchmarks" component for bugs and https://chromium.googlesource.com/catapult for downloading and editing source code..
https://chromium.googlesource.com/catapult
BSD 3-Clause "New" or "Revised" License
1.91k stars 563 forks source link

Dashboard - Investigate why resource_sizes alerts aren't showing up on graphs #4518

Closed simonhatch closed 6 years ago

simonhatch commented 6 years ago

From https://bugs.chromium.org/p/chromium/issues/detail?id=831328

Don't see anything specifically wrong, sheriff and anomaly config are set, /debug_alert implies we should see an alert, but no alert.

https://chromeperf.appspot.com/debug_alert?test_path=ChromiumPerf%2FAndroid+Builder+Perf%2Fresource_sizes+%28MonochromePublic.apk%29%2FMonochromePublic.apk_Specifics%2Fnormalized+apk+size&rev=561555&num_before=200&num_after=200&config=%7B%0D%0A++%22max_window_size%22%3A+2%2C+%0D%0A++%22min_absolute_change%22%3A+16000%2C+%0D%0A++%22min_relative_change%22%3A+0%2C+%0D%0A++%22min_segment_size%22%3A+0%0D%0A%7D

There's also supposed to be an alert on this graph, https://chromeperf.appspot.com/group_report?keys=agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgICQ6f34vwkM, pulled it from the alerts table but there's nothing showing

@anniesullie @eakuefner @benshayden

simonhatch commented 6 years ago

Update, the alert actually exists in datastore, but we're not surfacing it for some reason. I'm suspecting the recent QueryAsync changes @benshayden