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

Side by Side Diff: slave/skia_slave_scripts/compare_gms.py

Issue 316333002: display "latest GM failures" link in its own line under build step Base URL: https://skia.googlesource.com/buildbot.git@master
Patch Set: Created 6 years, 6 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
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 2 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """ Compare the generated GM images to the baselines """ 6 """ Compare the generated GM images to the baselines """
7 7
8 # System-level imports 8 # System-level imports
9 import os 9 import os
10 import sys 10 import sys
(...skipping 13 matching lines...) Expand all
24 json_summary_path = misc.GetAbsPath(os.path.join( 24 json_summary_path = misc.GetAbsPath(os.path.join(
25 self._gm_actual_dir, run_gm.JSON_SUMMARY_FILENAME)) 25 self._gm_actual_dir, run_gm.JSON_SUMMARY_FILENAME))
26 26
27 # Temporary list of builders who are allowed to fail this step without the 27 # Temporary list of builders who are allowed to fail this step without the
28 # bot turning red. 28 # bot turning red.
29 may_fail_with_warning = [] 29 may_fail_with_warning = []
30 # This import must happen after BuildStep.__init__ because it requires that 30 # This import must happen after BuildStep.__init__ because it requires that
31 # CWD is in PYTHONPATH, and BuildStep.__init__ may change the CWD. 31 # CWD is in PYTHONPATH, and BuildStep.__init__ may change the CWD.
32 from gm import display_json_results 32 from gm import display_json_results
33 success = display_json_results.Display(json_summary_path) 33 success = display_json_results.Display(json_summary_path)
34 print ('%s<a href="%s?resultsToLoad=/results/failures&builder=%s">' 34 print ('%s%s?resultsToLoad=/results/failures&builder=%s' % (
35 'link</a>' % ( 35 skia_vars.GetGlobalVariable('latest_gm_failures_preamble'),
36 skia_vars.GetGlobalVariable('latest_gm_failures_preamble'), 36 LIVE_REBASELINE_SERVER_BASEURL, self._builder_name))
37 LIVE_REBASELINE_SERVER_BASEURL, self._builder_name))
38 if not success: 37 if not success:
39 if self._builder_name in may_fail_with_warning: 38 if self._builder_name in may_fail_with_warning:
40 raise BuildStepWarning('Expectations mismatch in %s!' % 39 raise BuildStepWarning('Expectations mismatch in %s!' %
41 json_summary_path) 40 json_summary_path)
42 else: 41 else:
43 raise Exception('Expectations mismatch in %s!' % json_summary_path) 42 raise Exception('Expectations mismatch in %s!' % json_summary_path)
44 43
45 44
46 if '__main__' == __name__: 45 if '__main__' == __name__:
47 sys.exit(BuildStep.RunBuildStep(CompareGMs)) 46 sys.exit(BuildStep.RunBuildStep(CompareGMs))
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698