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

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

Issue 2411893002: [Findit] Adding metadata for manually vs automatically triggered flake analyses (Closed)
Patch Set: Addressing comments Created 4 years, 2 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/master_flake_analysis.py
diff --git a/appengine/findit/model/flake/master_flake_analysis.py b/appengine/findit/model/flake/master_flake_analysis.py
index fde544cd7eb65a721c2702a44e4ce5c7b747a25d..24e3cf8a48e1da3132afdf0f544982df94b66899 100644
--- a/appengine/findit/model/flake/master_flake_analysis.py
+++ b/appengine/findit/model/flake/master_flake_analysis.py
@@ -6,7 +6,6 @@ import base64
from google.appengine.ext import ndb
-from model import analysis_status
from model.base_analysis import BaseAnalysis
from model.base_build_model import BaseBuildModel
from model.flake.flake_swarming_task import FlakeSwarmingTaskData
@@ -112,3 +111,11 @@ class MasterFlakeAnalysis(BaseAnalysis, BaseBuildModel, VersionedModel):
# The data points used to plot the flakiness graph build over build.
data_points = ndb.LocalStructuredProperty(
DataPoint, repeated=True, compressed=True)
+
+ # Whether the analysis was triggered by a manual request through check flake,
+ # Findit's automatic analysis upon detection, or Findit API.
+ triggering_source = ndb.IntegerProperty(default=None, indexed=True)
+
+ # Who triggered the analysis. Used for differentiating between manual and
+ # automatic runs, and determining the most active users to gather feedback.
+ triggering_user_email = ndb.StringProperty(default=None, indexed=False)
« no previous file with comments | « appengine/findit/handlers/flake/check_flake.py ('k') | appengine/findit/waterfall/flake/flake_analysis_service.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698