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

Issue 884984 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 17
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

[Findit] Flake Analyzer - Remove unused fields from master_flake_analysis

Project Member Reported by lijeffrey@chromium.org, Sep 18

Issue description

Remove dead fields from master_flake_analysis
 
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 18

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

commit 683d05a7b09e6fa7ee8d5efbbd0a63e4d792cfed
Author: Jeffrey Li <lijeffrey@chromium.org>
Date: Tue Sep 18 18:57:19 2018

[Findit] Flake Analyzer - Cleanup ununsed fields in MasterFlakeAnalysis

Cleanup dead code from MasterFlakeAnalysis
1. algorithm_parameters - not used anywhere
2. try_job_status - obsolete since merging pipelines
3. All associated code removed

Bug:  884984 
Change-Id: If6c491e2dc6d4518a37b084c18a6cfb148b940be
Reviewed-on: https://chromium-review.googlesource.com/1229518
Reviewed-by: Yuke Liao <liaoyuke@chromium.org>
Commit-Queue: Jeffrey Li <lijeffrey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#17672}
[modify] https://crrev.com/683d05a7b09e6fa7ee8d5efbbd0a63e4d792cfed/appengine/findit/services/flake_failure/flake_analysis_util.py
[modify] https://crrev.com/683d05a7b09e6fa7ee8d5efbbd0a63e4d792cfed/appengine/findit/handlers/flake/list_flakes.py
[modify] https://crrev.com/683d05a7b09e6fa7ee8d5efbbd0a63e4d792cfed/appengine/findit/model/flake/analysis/master_flake_analysis.py
[modify] https://crrev.com/683d05a7b09e6fa7ee8d5efbbd0a63e4d792cfed/appengine/findit/services/flake_failure/test/flake_analysis_util_test.py
[modify] https://crrev.com/683d05a7b09e6fa7ee8d5efbbd0a63e4d792cfed/appengine/findit/services/flake_failure/test/flake_report_util_test.py
[modify] https://crrev.com/683d05a7b09e6fa7ee8d5efbbd0a63e4d792cfed/appengine/findit/handlers/flake/check_flake.py
[modify] https://crrev.com/683d05a7b09e6fa7ee8d5efbbd0a63e4d792cfed/appengine/findit/services/flake_failure/test/culprit_util_test.py
[modify] https://crrev.com/683d05a7b09e6fa7ee8d5efbbd0a63e4d792cfed/appengine/findit/waterfall/flake/initialize_flake_pipeline.py
[modify] https://crrev.com/683d05a7b09e6fa7ee8d5efbbd0a63e4d792cfed/appengine/findit/model/flake/analysis/test/master_flake_analysis_test.py
[modify] https://crrev.com/683d05a7b09e6fa7ee8d5efbbd0a63e4d792cfed/appengine/findit/handlers/flake/test/check_flake_test.py
[modify] https://crrev.com/683d05a7b09e6fa7ee8d5efbbd0a63e4d792cfed/appengine/findit/templates/flake/result.html

Status: Fixed (was: Started)

Sign in to add a comment