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

Issue 685448 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 676470



Sign in to add a comment

[Predator] Add ``ClassifySuspect`` to component classifier and project classifier.

Project Member Reported by kateso...@chromium.org, Jan 26 2017

Issue description

[Predator] Add ``ClassifySuspect`` to component classifier and project classifier.

In order to implement ``TouchCrashedComponent`` and ``TouchCrashedProject`` features in a clean way, we need ``ClassifySuspect`` in both component classifier and project classifier.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 30 2017

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

commit c6075535547c4e038d4466680d90479388581625
Author: katesonia <katesonia@chromium.org>
Date: Mon Jan 30 22:33:59 2017

[Predator] Add ``Project`` class and ``ClassifySuspect`` method to project classifier and component classifier.

Add ``Project`` class and ``ClassifySuspect`` method to project classifier and component classifier.

Also add validation for component classifier and project classifier.

N.B. Since I changed the config format of project classifier, so I included the validations in this cl.

BUG= 657177 ,  685448 
TBR=wrengr@chromium.org

Review-Url: https://codereview.chromium.org/2657913002

[modify] https://crrev.com/c6075535547c4e038d4466680d90479388581625/appengine/findit/crash/component.py
[modify] https://crrev.com/c6075535547c4e038d4466680d90479388581625/appengine/findit/crash/component_classifier.py
[modify] https://crrev.com/c6075535547c4e038d4466680d90479388581625/appengine/findit/crash/findit_for_chromecrash.py
[modify] https://crrev.com/c6075535547c4e038d4466680d90479388581625/appengine/findit/crash/predator.py
[add] https://crrev.com/c6075535547c4e038d4466680d90479388581625/appengine/findit/crash/project.py
[modify] https://crrev.com/c6075535547c4e038d4466680d90479388581625/appengine/findit/crash/project_classifier.py
[modify] https://crrev.com/c6075535547c4e038d4466680d90479388581625/appengine/findit/crash/test/component_classifier_test.py
[modify] https://crrev.com/c6075535547c4e038d4466680d90479388581625/appengine/findit/crash/test/findit_for_chromecrash_test.py
[modify] https://crrev.com/c6075535547c4e038d4466680d90479388581625/appengine/findit/crash/test/predator_testcase.py
[modify] https://crrev.com/c6075535547c4e038d4466680d90479388581625/appengine/findit/crash/test/project_classifier_test.py
[add] https://crrev.com/c6075535547c4e038d4466680d90479388581625/appengine/findit/crash/test/project_test.py
[modify] https://crrev.com/c6075535547c4e038d4466680d90479388581625/appengine/findit/handlers/crash/crash_config.py
[add] https://crrev.com/c6075535547c4e038d4466680d90479388581625/appengine/findit/handlers/crash/test/crash_config_test.py
[modify] https://crrev.com/c6075535547c4e038d4466680d90479388581625/appengine/findit/libs/gitiles/change_log.py
[modify] https://crrev.com/c6075535547c4e038d4466680d90479388581625/appengine/findit/libs/gitiles/test/change_log_test.py

Status: Fixed (was: Assigned)

Sign in to add a comment