[Findit] Findit integretion with Cracas and Clusterfuzz Based on design doc https://docs.google.com/a/google.com/document/d/1XkGjz7O2WNoGz2_x2JpbaCFZKsKqYpeLy9LiwfgKsZA/edit?usp=sharing Refactor findit pipeline to integrate with Cracas and Clusterfuzz.
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/bdee0f60631fbffc47f4ca7c58b9b1241da8af86 commit bdee0f60631fbffc47f4ca7c58b9b1241da8af86 Author: katesonia <katesonia@chromium.org> Date: Wed Sep 21 18:42:41 2016 [Findit] Add findit_for_client to do analysis based on client_id Add findit_for_client on top of findit_for_fracas and findit_for_clusterfuzz(to be implemented) to do client-specific checking and call different module to do analysis based on client_id. Doc: https://docs.google.com/a/google.com/document/d/1XkGjz7O2WNoGz2_x2JpbaCFZKsKqYpeLy9LiwfgKsZA/edit?usp=sharing BUG= 645203 Review-Url: https://codereview.chromium.org/2299883005 [modify] https://crrev.com/bdee0f60631fbffc47f4ca7c58b9b1241da8af86/appengine/findit/common/appengine_util.py [modify] https://crrev.com/bdee0f60631fbffc47f4ca7c58b9b1241da8af86/appengine/findit/common/constants.py [modify] https://crrev.com/bdee0f60631fbffc47f4ca7c58b9b1241da8af86/appengine/findit/crash/crash_pipeline.py [rename] https://crrev.com/bdee0f60631fbffc47f4ca7c58b9b1241da8af86/appengine/findit/crash/findit_for_chromecrash.py [add] https://crrev.com/bdee0f60631fbffc47f4ca7c58b9b1241da8af86/appengine/findit/crash/findit_for_client.py [modify] https://crrev.com/bdee0f60631fbffc47f4ca7c58b9b1241da8af86/appengine/findit/crash/test/crash_pipeline_test.py [rename] https://crrev.com/bdee0f60631fbffc47f4ca7c58b9b1241da8af86/appengine/findit/crash/test/findit_for_chromecrash_test.py [add] https://crrev.com/bdee0f60631fbffc47f4ca7c58b9b1241da8af86/appengine/findit/crash/test/findit_for_client_test.py [modify] https://crrev.com/bdee0f60631fbffc47f4ca7c58b9b1241da8af86/appengine/findit/crash/type_enums.py [modify] https://crrev.com/bdee0f60631fbffc47f4ca7c58b9b1241da8af86/appengine/findit/handlers/crash/crash_handler.py [modify] https://crrev.com/bdee0f60631fbffc47f4ca7c58b9b1241da8af86/appengine/findit/handlers/crash/test/crash_handler_test.py [modify] https://crrev.com/bdee0f60631fbffc47f4ca7c58b9b1241da8af86/appengine/findit/model/crash/crash_config.py [modify] https://crrev.com/bdee0f60631fbffc47f4ca7c58b9b1241da8af86/appengine/findit/model/crash/test/crash_config_test.py
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/aaedba6f2626f2493ba5b7a2814bf16ba6806799 commit aaedba6f2626f2493ba5b7a2814bf16ba6806799 Author: katesonia <katesonia@chromium.org> Date: Wed Sep 21 20:06:06 2016 [Findit] Add cracas_crash_analysis Add CracasCrashAnalysis, the model is identical as FracasCrashAnalysis, so just make a copy to help findit seperate analysis from Fracas and Cracas in database. BUG= 645203 Review-Url: https://codereview.chromium.org/2322133002 [modify] https://crrev.com/aaedba6f2626f2493ba5b7a2814bf16ba6806799/appengine/findit/crash/findit_for_client.py [add] https://crrev.com/aaedba6f2626f2493ba5b7a2814bf16ba6806799/appengine/findit/model/crash/chrome_crash_analysis.py [add] https://crrev.com/aaedba6f2626f2493ba5b7a2814bf16ba6806799/appengine/findit/model/crash/cracas_crash_analysis.py [modify] https://crrev.com/aaedba6f2626f2493ba5b7a2814bf16ba6806799/appengine/findit/model/crash/crash_analysis.py [modify] https://crrev.com/aaedba6f2626f2493ba5b7a2814bf16ba6806799/appengine/findit/model/crash/fracas_crash_analysis.py [add] https://crrev.com/aaedba6f2626f2493ba5b7a2814bf16ba6806799/appengine/findit/model/crash/test/chrome_crash_analysis_test.py [add] https://crrev.com/aaedba6f2626f2493ba5b7a2814bf16ba6806799/appengine/findit/model/crash/test/cracas_crash_analysis_test.py [modify] https://crrev.com/aaedba6f2626f2493ba5b7a2814bf16ba6806799/appengine/findit/model/crash/test/crash_analysis_test.py [modify] https://crrev.com/aaedba6f2626f2493ba5b7a2814bf16ba6806799/appengine/findit/model/crash/test/fracas_crash_analysis_test.py
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/c2a5bf2e88339240308928d1f7c18faeda2d939a commit c2a5bf2e88339240308928d1f7c18faeda2d939a Author: katesonia <katesonia@chromium.org> Date: Wed Sep 21 21:18:14 2016 [Findit] Add cracas config in config page. New config page: https://findit-for-me.googleplex.com/crash/config BUG= 645203 Review-Url: https://codereview.chromium.org/2356673002 [modify] https://crrev.com/c2a5bf2e88339240308928d1f7c18faeda2d939a/appengine/findit/handlers/crash/crash_config.py [modify] https://crrev.com/c2a5bf2e88339240308928d1f7c18faeda2d939a/appengine/findit/model/crash/crash_config.py [modify] https://crrev.com/c2a5bf2e88339240308928d1f7c18faeda2d939a/appengine/findit/templates/crash/crash_config.html
Comment 1 by bugdroid1@chromium.org
, Sep 21 2016