Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1341)

Unified Diff: appengine/findit/model/flake/flake_culprit.py

Issue 2608853002: [Findit] Adding suspected CL to master flake analysis in preparation for try jobs (Closed)
Patch Set: Addressing comment Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: appengine/findit/model/flake/flake_culprit.py
diff --git a/appengine/findit/model/flake/flake_culprit.py b/appengine/findit/model/flake/flake_culprit.py
new file mode 100644
index 0000000000000000000000000000000000000000..5149cfb45465053fc65c3daae13223bc60fa08f5
--- /dev/null
+++ b/appengine/findit/model/flake/flake_culprit.py
@@ -0,0 +1,26 @@
+# Copyright 2017 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+from google.appengine.ext import ndb
+
+from model.base_suspected_cl import BaseSuspectedCL
+
+
+class FlakeCulprit(BaseSuspectedCL):
+ """Represents information about a culprit CL for flake analysis."""
+
+ # Triage status for whether the CL is correct or not.
+ status = ndb.IntegerProperty(indexed=True, default=None)
+
+ # Url to the code review or change log associated with this change.
+ url = ndb.StringProperty(indexed=False)
+
+ # Arguments number differs from overridden method - pylint: disable=W0221
+ @classmethod
+ def Create(
+ cls, repo_name, revision, commit_position, url): # pragma: no cover
+ instance = super(FlakeCulprit, cls).Create(
+ repo_name, revision, commit_position)
+ instance.url = url
+ return instance
« no previous file with comments | « appengine/findit/handlers/flake/test/check_flake_test.py ('k') | appengine/findit/model/flake/flake_try_job_data.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698