Several hard-coded parameters have been introduced in Findit that can be moved to config.
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/1fb3937ec435e057aa87becf78d8f33e6bb45071 commit 1fb3937ec435e057aa87becf78d8f33e6bb45071 Author: lijeffrey <lijeffrey@chromium.org> Date: Mon Aug 22 22:06:48 2016 [Findit] Move configurable parameters in try_job_util.py to config BUG= 639542 Review-Url: https://codereview.chromium.org/2258373002 [modify] https://crrev.com/1fb3937ec435e057aa87becf78d8f33e6bb45071/appengine/findit/handlers/config.py [modify] https://crrev.com/1fb3937ec435e057aa87becf78d8f33e6bb45071/appengine/findit/handlers/test/config_test.py [modify] https://crrev.com/1fb3937ec435e057aa87becf78d8f33e6bb45071/appengine/findit/waterfall/test/waterfall_config_test.py [modify] https://crrev.com/1fb3937ec435e057aa87becf78d8f33e6bb45071/appengine/findit/waterfall/test/wf_testcase.py [modify] https://crrev.com/1fb3937ec435e057aa87becf78d8f33e6bb45071/appengine/findit/waterfall/try_job_util.py
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/b1474528edcca30a7b4bfc7d16e725157bc764cd commit b1474528edcca30a7b4bfc7d16e725157bc764cd Author: lijeffrey <lijeffrey@chromium.org> Date: Fri Aug 26 04:18:26 2016 [Findit] Moving check flake parameters to config BUG= 639542 Review-Url: https://codereview.chromium.org/2272953002 [modify] https://crrev.com/b1474528edcca30a7b4bfc7d16e725157bc764cd/appengine/findit/handlers/config.py [modify] https://crrev.com/b1474528edcca30a7b4bfc7d16e725157bc764cd/appengine/findit/handlers/test/config_test.py [modify] https://crrev.com/b1474528edcca30a7b4bfc7d16e725157bc764cd/appengine/findit/model/wf_config.py [modify] https://crrev.com/b1474528edcca30a7b4bfc7d16e725157bc764cd/appengine/findit/templates/config.html [modify] https://crrev.com/b1474528edcca30a7b4bfc7d16e725157bc764cd/appengine/findit/waterfall/flake/recursive_flake_pipeline.py [modify] https://crrev.com/b1474528edcca30a7b4bfc7d16e725157bc764cd/appengine/findit/waterfall/test/trigger_flake_swarming_task_pipeline_test.py [modify] https://crrev.com/b1474528edcca30a7b4bfc7d16e725157bc764cd/appengine/findit/waterfall/test/waterfall_config_test.py [modify] https://crrev.com/b1474528edcca30a7b4bfc7d16e725157bc764cd/appengine/findit/waterfall/test/wf_testcase.py [modify] https://crrev.com/b1474528edcca30a7b4bfc7d16e725157bc764cd/appengine/findit/waterfall/trigger_base_swarming_task_pipeline.py [modify] https://crrev.com/b1474528edcca30a7b4bfc7d16e725157bc764cd/appengine/findit/waterfall/trigger_flake_swarming_task_pipeline.py [modify] https://crrev.com/b1474528edcca30a7b4bfc7d16e725157bc764cd/appengine/findit/waterfall/waterfall_config.py
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/7bdbd00739c9d32ec5d8be755cc7ba42d2261423 commit 7bdbd00739c9d32ec5d8be755cc7ba42d2261423 Author: lijeffrey <lijeffrey@chromium.org> Date: Fri Aug 26 21:51:28 2016 [Findit] Moving check flake max_builds_to_look_back to config BUG= 639542 Review-Url: https://codereview.chromium.org/2282093002 [modify] https://crrev.com/7bdbd00739c9d32ec5d8be755cc7ba42d2261423/appengine/findit/handlers/config.py [modify] https://crrev.com/7bdbd00739c9d32ec5d8be755cc7ba42d2261423/appengine/findit/handlers/test/config_test.py [modify] https://crrev.com/7bdbd00739c9d32ec5d8be755cc7ba42d2261423/appengine/findit/waterfall/flake/initialize_flake_pipeline.py [modify] https://crrev.com/7bdbd00739c9d32ec5d8be755cc7ba42d2261423/appengine/findit/waterfall/test/waterfall_config_test.py [modify] https://crrev.com/7bdbd00739c9d32ec5d8be755cc7ba42d2261423/appengine/findit/waterfall/test/wf_testcase.py
Comment 1 by bugdroid1@chromium.org
, Aug 22 2016