[Findit] Integrate with Fracas through Pub/Sub.

appengine/findit/model/crash/crash_analysis.py

Index: appengine/findit/model/crash/crash_analysis.py
diff --git a/appengine/findit/model/crash/crash_analysis.py b/appengine/findit/model/crash/crash_analysis.py
new file mode 100644
index 0000000000000000000000000000000000000000..0683fbcef46bcc6e434cec19cb20f2929b2ebd97
--- /dev/null
+++ b/appengine/findit/model/crash/crash_analysis.py
@@ -0,0 +1,67 @@
+# Copyright 2016 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 import analysis_status
+
+
+class CrashAnalysis(ndb.Model):
+  """Base class to represent an analysis of a Chrome crash."""
+  # In which version or revision of Chrome the crash occurred. Either a version
+  # number for Chrome build or a git commit hash/position for chromium build.
+  crashed_version = ndb.StringProperty(indexed=False)

[Martin Barbella, 2016/04/08 18:10:24]

Nit: whitespace between properties for readability.

[stgao, 2016/04/11 20:32:20]

Sounds good. I also have separators for different groups of properties.

+  # The stack_trace_string.
+  stack_trace = ndb.StringProperty(indexed=False)
+  # The signature of the crash.
+  signature = ndb.StringProperty(indexed=False)

[Sharu, 2016/04/12 00:23:14]

signature is fracas specific, clusterfuzz doesn't have signature, something
similar is the crash state, which usually has 3 top crash functions.

So should we put this in subclass?

[stgao, 2016/04/12 18:27:35]

From the perspective of abstraction, we could treat crash state in ClusterFuzz
as a crash signature. So it's fine to leave as is.

+
+  # The url path to the pipeline status page.
+  pipeline_status_path = ndb.StringProperty(indexed=False)
+  # The status of the analysis.
+  status = ndb.IntegerProperty(
+      default=analysis_status.PENDING, indexed=False)
+  # When the analysis was requested.
+  requested_time = ndb.DateTimeProperty(indexed=True)
+  # When the analysis was started.
+  started_time = ndb.DateTimeProperty(indexed=False)
+  # When the analysis was completed.
+  completed_time = ndb.DateTimeProperty(indexed=False)
+  # Which version of findit produces this result.
+  findit_version = ndb.StringProperty(indexed=False)
+
+  # Analysis results.
+  result = ndb.JsonProperty(compressed=True, indexed=False)
+
+  # Tags for query and monitoring.
+  has_regression_range = ndb.BooleanProperty(indexed=True)
+  found_suspects = ndb.BooleanProperty(indexed=True)
+  solution = ndb.StringProperty(indexed=True)  # 'core', 'blame', etc.
+
+  def Reset(self):
+    self.pipeline_status_path = None
+    self.status = analysis_status.PENDING
+    self.requested_time = None
+    self.started_time = None
+    self.completed_time = None
+    self.findit_version = None
+    self.has_regression_range = None
+    self.found_suspects = None
+    self.solution = None
+
+  @property
+  def completed(self):
+    return self.status in (
+        analysis_status.COMPLETED, analysis_status.ERROR)
+
+  @property
+  def failed(self):
+    return self.status == analysis_status.ERROR
+
+  @property
+  def duration(self):
+    if not self.completed:
+      return None
+
+    return int((self.completed_time - self.started_time).total_seconds())