1. Add subpipelines for different failure type 2. Use different wrapper to call sub pipelines separately.
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/9543de61b17e89346d4d56f22113e11f77c71309 commit 9543de61b17e89346d4d56f22113e11f77c71309 Author: Chan <chanli@chromium.org> Date: Wed Sep 20 17:00:57 2017 [Findit] Add new sub pipelines for detecting first failure based on failure type. This CL is for refactoring. Bug: 764531 Change-Id: I866f7f0d48929d4d06cac4ed888884f6bae8fdea Reviewed-on: https://chromium-review.googlesource.com/673871 Commit-Queue: Chan Li <chanli@chromium.org> Reviewed-by: Shuotao Gao <stgao@chromium.org> [add] https://crrev.com/9543de61b17e89346d4d56f22113e11f77c71309/appengine/findit/pipelines/compile_failure/detect_first_compile_failure_pipeline.py [add] https://crrev.com/9543de61b17e89346d4d56f22113e11f77c71309/appengine/findit/pipelines/compile_failure/test/detect_first_compile_failure_pipeline_test.py [add] https://crrev.com/9543de61b17e89346d4d56f22113e11f77c71309/appengine/findit/pipelines/test_failure/test/__init__.py [add] https://crrev.com/9543de61b17e89346d4d56f22113e11f77c71309/appengine/findit/pipelines/test_failure/test/detect_first_test_failure_pipeline_test.py [add] https://crrev.com/9543de61b17e89346d4d56f22113e11f77c71309/appengine/findit/pipelines/test_failure/detect_first_test_failure_pipeline.py [add] https://crrev.com/9543de61b17e89346d4d56f22113e11f77c71309/appengine/findit/pipelines/compile_failure/test/__init__.py
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/4c3c4ee823dd38b0320b414ae455d60e49cf6fcb commit 4c3c4ee823dd38b0320b414ae455d60e49cf6fcb Author: Chan <chanli@chromium.org> Date: Wed Sep 20 17:53:26 2017 [Findit] Refactor: Add sub pipelines for extracting signals based on failure type. Bug: 764531 Change-Id: If44a5be19640b06471fdeac91f3ef9eebe68f7f1 Reviewed-on: https://chromium-review.googlesource.com/673983 Commit-Queue: Chan Li <chanli@chromium.org> Reviewed-by: Shuotao Gao <stgao@chromium.org> [add] https://crrev.com/4c3c4ee823dd38b0320b414ae455d60e49cf6fcb/appengine/findit/pipelines/compile_failure/extract_signal_for_compile_pipeline.py [add] https://crrev.com/4c3c4ee823dd38b0320b414ae455d60e49cf6fcb/appengine/findit/pipelines/test_failure/extract_signal_for_test_pipeline.py [add] https://crrev.com/4c3c4ee823dd38b0320b414ae455d60e49cf6fcb/appengine/findit/pipelines/test_failure/test/extract_signal_for_test_pipeline_test.py [add] https://crrev.com/4c3c4ee823dd38b0320b414ae455d60e49cf6fcb/appengine/findit/pipelines/compile_failure/test/extract_signal_for_compile_pipeline_test.py
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/2f737fd6715f62eae0be3cad59c5cb8f98911485 commit 2f737fd6715f62eae0be3cad59c5cb8f98911485 Author: Chan <chanli@chromium.org> Date: Thu Sep 21 17:23:39 2017 [Findit] Refactor: Add subpipelines to identify suspects based on failure type. Bug: 757664 , 764531 Change-Id: I2a6c92c3c3584f2cf2ed36cb8f8bd310af169c73 Reviewed-on: https://chromium-review.googlesource.com/674025 Commit-Queue: Chan Li <chanli@chromium.org> Reviewed-by: Shuotao Gao <stgao@chromium.org> [add] https://crrev.com/2f737fd6715f62eae0be3cad59c5cb8f98911485/appengine/findit/pipelines/test_failure/test/identify_test_failure_suspect_pipeline_test.py [add] https://crrev.com/2f737fd6715f62eae0be3cad59c5cb8f98911485/appengine/findit/pipelines/compile_failure/test/identify_compile_failure_suspect_pipeline_test.py [add] https://crrev.com/2f737fd6715f62eae0be3cad59c5cb8f98911485/appengine/findit/pipelines/compile_failure/identify_compile_failure_suspect_pipeline.py [add] https://crrev.com/2f737fd6715f62eae0be3cad59c5cb8f98911485/appengine/findit/pipelines/test_failure/identify_test_failure_suspect_pipeline.py
Comment 1 by bugdroid1@chromium.org
, Sep 20 2017