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

Issue 828480 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Sep 4
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

[Findit] Allow autorevert for flaky tests to be turned off/on with the config

Project Member Reported by wylieb@chromium.org, Apr 3 2018

Issue description

Rely on false for the default values.
 
Labels: -Pri-3 Pri-1
Owner: wylieb@chromium.org
Status: Assigned (was: Available)
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 10 2018

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

commit 75ede494b3d5ed4fa9077e8418c189f942ec195d
Author: Brandon Wylie <wylieb@chromium.org>
Date: Tue Apr 10 00:16:46 2018

[Findit] Flake Analyzer - Fit and finish for flake autoreverts

* Adding support for Flaky test: step.test when creating a CL. This
  is for the audit-app.
* Adding consistent arguments for codereview class.
* Adding optional bug_id field and optional footer for revert reason.
* Adding config ovverride to enable autorevert.

Bug: 826870,  828480 
Change-Id: I9f620220f54aee8e300efd66174568d63dcd23ee
Reviewed-on: https://chromium-review.googlesource.com/999968
Reviewed-by: Shuotao Gao <stgao@chromium.org>
Commit-Queue: Brandon Wylie <wylieb@chromium.org>

[modify] https://crrev.com/75ede494b3d5ed4fa9077e8418c189f942ec195d/appengine/findit/handlers/test/config_test.py
[modify] https://crrev.com/75ede494b3d5ed4fa9077e8418c189f942ec195d/appengine/findit/services/gerrit.py
[modify] https://crrev.com/75ede494b3d5ed4fa9077e8418c189f942ec195d/appengine/findit/services/test/gerrit_test.py
[modify] https://crrev.com/75ede494b3d5ed4fa9077e8418c189f942ec195d/appengine/findit/pipelines/test/create_revert_cl_pipeline_test.py
[modify] https://crrev.com/75ede494b3d5ed4fa9077e8418c189f942ec195d/appengine/findit/infra_api_clients/codereview/rietveld.py
[modify] https://crrev.com/75ede494b3d5ed4fa9077e8418c189f942ec195d/appengine/findit/services/flake_failure/culprit_util.py
[modify] https://crrev.com/75ede494b3d5ed4fa9077e8418c189f942ec195d/appengine/findit/infra_api_clients/codereview/gerrit.py
[modify] https://crrev.com/75ede494b3d5ed4fa9077e8418c189f942ec195d/appengine/findit/services/flake_failure/test/culprit_util_test.py
[modify] https://crrev.com/75ede494b3d5ed4fa9077e8418c189f942ec195d/appengine/findit/infra_api_clients/codereview/codereview.py
[modify] https://crrev.com/75ede494b3d5ed4fa9077e8418c189f942ec195d/appengine/findit/handlers/config.py

Comment 3 by wylieb@chromium.org, Apr 20 2018

Owner: ----
Status: Available (was: Assigned)
Status: Fixed (was: Available)
Auto revert for flake analyzer is now configurable

Sign in to add a comment