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

Side by Side Diff: appengine/chromium_try_flakes/handlers/flake_issues.py

Issue 1934403002: Mark taskkill as infra step (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: Created 4 years, 7 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 """Task queue endpoints for creating and updating issues on issue tracker.""" 5 """Task queue endpoints for creating and updating issues on issue tracker."""
6 6
7 import datetime 7 import datetime
8 import json 8 import json
9 import logging 9 import logging
10 import urllib2 10 import urllib2
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
72 NUM_TIMES_IN_QUEUE_FOR_STALENESS = 5 72 NUM_TIMES_IN_QUEUE_FOR_STALENESS = 5
73 VERY_STALE_FLAKES_MESSAGE_MANY_TIMES = ( 73 VERY_STALE_FLAKES_MESSAGE_MANY_TIMES = (
74 'Reporting to stale-flakes-reports@google.com to investigate why this ' 74 'Reporting to stale-flakes-reports@google.com to investigate why this '
75 'issue has been in the appropriate queue %d times or more.' % 75 'issue has been in the appropriate queue %d times or more.' %
76 NUM_TIMES_IN_QUEUE_FOR_STALENESS) 76 NUM_TIMES_IN_QUEUE_FOR_STALENESS)
77 STALE_FLAKES_ML = 'stale-flakes-reports@google.com' 77 STALE_FLAKES_ML = 'stale-flakes-reports@google.com'
78 MAX_GAP_FOR_FLAKINESS_PERIOD = datetime.timedelta(days=3) 78 MAX_GAP_FOR_FLAKINESS_PERIOD = datetime.timedelta(days=3)
79 KNOWN_TROOPER_FLAKE_NAMES = [ 79 KNOWN_TROOPER_FLAKE_NAMES = [
80 'analyze', 'bot_update', 'compile (with patch)', 'compile', 80 'analyze', 'bot_update', 'compile (with patch)', 'compile',
81 'device_status_check', 'gclient runhooks (with patch)', 'Patch', 81 'device_status_check', 'gclient runhooks (with patch)', 'Patch',
82 'process_dumps', 'provision_devices', 'update_scripts'] 82 'process_dumps', 'provision_devices', 'update_scripts', 'taskkill']
83 83
84 # Flakes in these steps are always ignored: 84 # Flakes in these steps are always ignored:
85 # - steps: always red when any other step is red (duplicates failure) 85 # - steps: always red when any other step is red (duplicates failure)
86 # - presubmit: typically red due to missing OWNERs LGTM, not a flake 86 # - presubmit: typically red due to missing OWNERs LGTM, not a flake
87 # - recipe failure reason: always red when build fails (not a failure) 87 # - recipe failure reason: always red when build fails (not a failure)
88 # - test results: always red when another step is red (not a failure) 88 # - test results: always red when another step is red (not a failure)
89 # - Uncaught Exception: summary step referring to an exception in another 89 # - Uncaught Exception: summary step referring to an exception in another
90 # step (duplicates failure) 90 # step (duplicates failure)
91 # There are additional rules for non-trivial cases in the FlakyRun.post method. 91 # There are additional rules for non-trivial cases in the FlakyRun.post method.
92 IGNORED_STEPS = ['steps', 'presubmit', 'recipe failure reason', 'test results', 92 IGNORED_STEPS = ['steps', 'presubmit', 'recipe failure reason', 'test results',
(...skipping 496 matching lines...) Expand 10 before | Expand all | Expand 10 after
589 failure_run.buildnumber, step) 589 failure_run.buildnumber, step)
590 for flake in flakes: 590 for flake in flakes:
591 flake_occurrence = FlakeOccurrence(name=step_name, failure=flake) 591 flake_occurrence = FlakeOccurrence(name=step_name, failure=flake)
592 flaky_run.flakes.append(flake_occurrence) 592 flaky_run.flakes.append(flake_occurrence)
593 flakes_to_update.append(flake) 593 flakes_to_update.append(flake)
594 594
595 flaky_run_key = flaky_run.put() 595 flaky_run_key = flaky_run.put()
596 for flake in flakes_to_update: 596 for flake in flakes_to_update:
597 self.add_failure_to_flake(flake, flaky_run_key, failure_time) 597 self.add_failure_to_flake(flake, flaky_run_key, failure_time)
598 self.flaky_runs.increment_by(1) 598 self.flaky_runs.increment_by(1)
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698