New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 901912 link

Starred by 1 user
Status: Fixed
Owner:
lijeffrey@chromium.org
Closed: Nov 8
Cc:
█ robert...@chromium.org
chanli@chromium.org
lijeffrey@chromium.org
liaoyuke@chromium.org
st...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug

Blocking:
issue 901910



Sign in to add a comment

[Findit] Flake Analyzer / Flake Detection - Unify bug limits in config

Project Member Reported by lijeffrey@chromium.org, Nov 5 
Flake Analyzer and Flake Detection both have constants for limits on how many daily bug operations can be performed. There are 2 separate values (which are used in the same way), 1 for each Flake Analyzer and Flake Detector. These values should be merged and placed under Action Settings.

Comment 1 by lijeffrey@chromium.org, Nov 5

Components: Tools>Test>FindIt>Flakiness
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 6 

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/b1610f082f83e8b703344318b2a4d8267cd518e0

commit b1610f082f83e8b703344318b2a4d8267cd518e0
Author: Jeffrey Li <lijeffrey@chromium.org>
Date: Tue Nov 06 21:41:13 2018

[Findit] Flake Analyzer + Detection: Unify bug filing config

1. Move maximum number of bug updates to config, under action settings
2. Remove configs for whether or not to update bugs entirely: Instead
   just change the maximum number of bug updates to 0.

Bug:  901912 
Change-Id: I990edfaf4b31c8877329b7bc12c88d17e8123c20
Reviewed-on: https://chromium-review.googlesource.com/c/1318265
Reviewed-by: Chan Li <chanli@chromium.org>
Commit-Queue: Jeffrey Li <lijeffrey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#18820}
[modify] https://crrev.com/b1610f082f83e8b703344318b2a4d8267cd518e0/appengine/findit/handlers/test/config_test.py
[modify] https://crrev.com/b1610f082f83e8b703344318b2a4d8267cd518e0/appengine/findit/handlers/config.py
[modify] https://crrev.com/b1610f082f83e8b703344318b2a4d8267cd518e0/appengine/findit/waterfall/test/waterfall_config_test.py
[modify] https://crrev.com/b1610f082f83e8b703344318b2a4d8267cd518e0/appengine/findit/waterfall/test/wf_testcase.py
[modify] https://crrev.com/b1610f082f83e8b703344318b2a4d8267cd518e0/appengine/findit/services/flake_failure/test/flake_report_util_test.py
[modify] https://crrev.com/b1610f082f83e8b703344318b2a4d8267cd518e0/appengine/findit/services/flake_detection/test/flake_report_util_test.py
[modify] https://crrev.com/b1610f082f83e8b703344318b2a4d8267cd518e0/appengine/findit/services/flake_failure/flake_report_util.py
[modify] https://crrev.com/b1610f082f83e8b703344318b2a4d8267cd518e0/appengine/findit/services/flake_detection/flake_report_util.py
[modify] https://crrev.com/b1610f082f83e8b703344318b2a4d8267cd518e0/appengine/findit/model/wf_config.py
[modify] https://crrev.com/b1610f082f83e8b703344318b2a4d8267cd518e0/appengine/findit/services/flake_failure/flake_constants.py

Comment 3 by lijeffrey@chromium.org, Nov 8

Status: Fixed (was: Started)
►

Sign in to add a comment