FindIt breakages |
|||
Issue descriptionhttps://cluster-fuzz.appspot.com/testcase?key=5130952057815040 Analysis of failed component others is not supported by Findit. Suspected Project: others How are you monitoring the results of your changes, I have seen ton of breakages recently with FindIt.
,
Jul 25 2016
Sure, let me look into it.
,
Jul 25 2016
This is because when we cannot find a dep path match for a file path, we return default value 'others' instead of 'chromium'. It's used to be fine, however recently there are a lot of cases where the file paths are like 'third_party/...', 'cc/...' instead of 'src/third_party/...', 'src/cc' Set the default value to 'src/', 'chromium' can solve this. Sorry, since I didn't make any functional change in clusterfuzz for quite a while and I am busy with developing features of findit appengine, so I am not actively monitoring the results of findit for clusterfuzz.
,
Jul 25 2016
That is a bad hack to rely on in the first place. Recent sanitizer changes removed src/, we should find some other proper way to do this. Can you ask around with fellow TE which do ClusterFUzz triage on why they didn't notice this. +cc Manu expliticly.
,
Jul 25 2016
There is better way to do that(it's doing ok from the fracas crash results monitoring), actually when I rewrite this part in findit appengine (already integrated with Fracas, will be integrated with clusterfuzz as well), I changed the clusterfuzz method to be the same. The fix cl is as below: https://chromereviews.googleplex.com/476097013/
,
Jul 26 2016
New version deployed, the fix works :) |
|||
►
Sign in to add a comment |
|||
Comment 1 by infe...@chromium.org
, Jul 25 2016