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

Unified Diff: appengine/findit/model/wf_analysis.py

Issue 2139093002: [Findit] Trigger swarming tasks after detech_first_faliure_pipeline (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: . Created 4 years, 5 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/wf_analysis.py
diff --git a/appengine/findit/model/wf_analysis.py b/appengine/findit/model/wf_analysis.py
index 2838adea93c03ac67cdba4ecd42d05e4cdaf226f..e2317a551ae03403f9a885d88d0055c16f259255 100644
--- a/appengine/findit/model/wf_analysis.py
+++ b/appengine/findit/model/wf_analysis.py
@@ -24,8 +24,11 @@ class WfAnalysis(BaseBuildModel):
@staticmethod
def Create(master_name, builder_name, build_number): # pragma: no cover
- return WfAnalysis(
+ analysis = WfAnalysis(
key=WfAnalysis._CreateKey(master_name, builder_name, build_number))
+ if analysis.failure_result_map is None:
+ analysis.failure_result_map = {}
lijeffrey 2016/07/12 22:36:27 I think this can be simplified slightly with anal
chanli 2016/07/12 22:54:56 Done.
+ return analysis
@staticmethod
def Get(master_name, builder_name, build_number): # pragma: no cover

Powered by Google App Engine
This is Rietveld 408576698