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

Unified Diff: appengine/findit/waterfall/flake/flake_analysis_service.py

Issue 2456033002: [Findit] Using flake_analysis_service and add bug number to check flake UI (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/waterfall/flake/flake_analysis_service.py
diff --git a/appengine/findit/waterfall/flake/flake_analysis_service.py b/appengine/findit/waterfall/flake/flake_analysis_service.py
index cf6d4214db934fb0f2ef86f7cb79dfac48775d66..9ca5e6e8d46e58de06cfaead18274e6ac28f18c8 100644
--- a/appengine/findit/waterfall/flake/flake_analysis_service.py
+++ b/appengine/findit/waterfall/flake/flake_analysis_service.py
@@ -204,6 +204,7 @@ def ScheduleAnalysisForFlake(request, user_email, is_admin, triggering_source):
version_number, build_step = _CheckForNewAnalysis(request)
if version_number and build_step:
# A new analysis is needed.
+ # TODO(lijeffrey): Add support for the force flag to trigger a rerun.
logging.info('A new analysis is needed for: %s', build_step)
analysis = initialize_flake_pipeline.ScheduleAnalysisIfNeeded(
build_step.wf_master_name, build_step.wf_builder_name,

Powered by Google App Engine
This is Rietveld 408576698