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

Unified Diff: appengine/findit/waterfall/try_job_enums.py

Issue 1591003002: [Findit] Modify tryjob pipelines to trigger try jobs for test failure. (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: rebase Created 4 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/waterfall/try_job_enums.py
diff --git a/infra/tools/cq_stats/__main__.py b/appengine/findit/waterfall/try_job_enums.py
similarity index 50%
copy from infra/tools/cq_stats/__main__.py
copy to appengine/findit/waterfall/try_job_enums.py
index 615ca98482109deb82fe8b187040bebf29541ae2..8cb644af785108712939eb0f5f0d8f996238934e 100644
--- a/infra/tools/cq_stats/__main__.py
+++ b/appengine/findit/waterfall/try_job_enums.py
@@ -1,12 +1,14 @@
-#!/usr/bin/env python
# Copyright 2015 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.
-import sys
-from infra.tools.cq_stats import cq_stats
+class TryJobType():
+ type_compile = 'compile'
stgao 2016/01/28 19:33:52 IIRC, constant usually use upper-case instead of l
chanli 2016/01/28 22:40:12 Done.
+ type_test = 'test'
stgao 2016/01/28 19:33:52 It seems the 'type' in the var name is a bit redun
chanli 2016/01/28 22:40:12 Done.
-if __name__ == '__main__':
- sys.exit(cq_stats.main())
+class TryJobStatus():
stgao 2016/01/28 19:33:52 It seems duplicating common.buildbucket_client.Bui
chanli 2016/01/28 22:40:12 Done.
+ scheduled = 'SCHEDULED'
+ started = 'STARTED'
+ completed = 'COMPLETED'

Powered by Google App Engine
This is Rietveld 408576698