Permalink
Browse files

bug 812727 - make summary an optional action. r=rail

  • Loading branch information...
1 parent fb5e95a commit 096596244b387159b653ea11465a28e8981bfcf0 @escapewindow escapewindow committed Dec 20, 2012
@@ -46,5 +46,6 @@
"restore-objdir",
"add-locales",
"package-multi",
+ "summary",
],
}
@@ -33,7 +33,7 @@
"release_config_file": "buildbot-configs/mozilla/release-fennec-mozilla-release.py",
- "default_actions": ["clobber", "pull", "download", "repack", "upload-unsigned-bits"],
+ "default_actions": ["clobber", "pull", "download", "repack", "upload-unsigned-bits", "summary"],
# signing (optional)
"keystore": KEYSTORE,
@@ -38,7 +38,7 @@
"release_config_file": "buildbot-configs/mozilla/release-fennec-mozilla-release.py",
- "default_actions": ["clobber", "pull", "download", "repack", "upload-unsigned-bits"],
+ "default_actions": ["clobber", "pull", "download", "repack", "upload-unsigned-bits", "summary"],
# signing (optional)
"keystore": KEYSTORE,
@@ -99,6 +99,7 @@
"sign",
"verify-signatures",
"upload-signed-bits",
+ "summary",
],
"keystore": KEYSTORE,
@@ -102,6 +102,7 @@
"sign",
"verify-signatures",
"upload-signed-bits",
+ "summary",
],
"keystore": KEYSTORE,
@@ -73,6 +73,7 @@
"setup",
"repack",
"upload-repacks",
+ "summary",
],
# Mock
"mock_target": "mozilla-centos6-i386",
@@ -73,5 +73,6 @@
"setup",
"repack",
"upload-repacks",
+ "summary",
],
}
@@ -72,6 +72,7 @@
"setup",
"repack",
"upload-repacks",
+ "summary",
],
# Mock
@@ -72,5 +72,6 @@
"setup",
"repack",
"upload-repacks",
+ "summary",
],
}
@@ -716,8 +716,6 @@ def run(self):
Postflight is quick testing for success after an action.
- Run self.summary() at the end.
-
"""
self.dump_config()
for action in self.all_actions:
@@ -729,7 +727,6 @@ def run(self):
self._possibly_run_method("preflight_%s" % method_name)
self._possibly_run_method(method_name, error_if_missing=True)
self._possibly_run_method("postflight_%s" % method_name)
- self.summary()
self.copy_logs_to_upload_dir()
sys.exit(self.return_code)
@@ -103,7 +103,7 @@ def __init__(self, require_config_file=True):
'backup-objdir',
'restore-objdir',
'add-locales', 'package-multi',
- 'upload-multi'],
+ 'upload-multi', 'summary'],
require_config_file=require_config_file)
def clobber(self):
@@ -75,6 +75,7 @@ def __init__(self, require_config_file=False):
all_actions=[
'pull',
'build',
+ 'summary',
],
require_config_file=require_config_file,
@@ -368,9 +368,8 @@ def run_tests(self):
parser.append_tinderboxprint_line(suite_name)
self.buildbot_status(tbpl_status, level=log_level)
- self.add_summary("The %s suite: %s ran with return status: %s" %
- (suite_name, suite, tbpl_status),
- level=log_level)
+ self.log("The %s suite: %s ran with return status: %s" %
+ (suite_name, suite, tbpl_status), level=log_level)
if __name__ == '__main__':
emulatorTest = B2GEmulatorTest()
@@ -40,9 +40,12 @@ def __init__(self, require_config_file=False):
all_actions=['list-config-files',
'test-json-configs',
'test-python-configs',
+ 'summary',
],
default_actions=['test-json-configs',
- 'test-python-configs'],
+ 'test-python-configs',
+ 'summary',
+ ],
require_config_file=require_config_file)
def query_config_files(self):
@@ -351,9 +351,8 @@ def _run_category_suites(self, suite_category, preflight_run_method=None):
parser.append_tinderboxprint_line(suite_name)
self.buildbot_status(tbpl_status, level=log_level)
- self.add_summary("The %s suite: %s ran with return status: %s" %
- (suite_category, suite, tbpl_status),
- level=log_level)
+ self.log("The %s suite: %s ran with return status: %s" %
+ (suite_category, suite, tbpl_status), level=log_level)
else:
self.debug('There were no suites to run for %s' % suite_category)
@@ -260,9 +260,8 @@ def run_marionette(self):
self.info("TinderboxPrint: marionette<br/>%s\n" % tsummary)
- self.add_summary("Marionette exited with return code %s: %s" % (code,
- status),
- level=level)
+ self.log("Marionette exited with return code %s: %s" % (code, status),
+ level=level)
self.buildbot_status(tbpl_status)
@@ -110,6 +110,7 @@ def __init__(self, require_config_file=True):
"upload-repacks",
"create-nightly-snippets",
"upload-nightly-snippets",
+ "summary",
],
require_config_file=require_config_file
)
@@ -112,6 +112,7 @@ def __init__(self, require_config_file=True):
"upload-unsigned-bits",
"sign",
"upload-signed-bits",
+ "summary",
],
require_config_file=require_config_file
)
@@ -73,11 +73,13 @@ def __init__(self, require_config_file=False):
# TODO
# 'upload',
# 'notify',
+ 'summary',
],
default_actions=['preclean',
'pull',
'download',
'run-mozmill',
+ 'summary',
],
require_config_file=require_config_file,
config={"virtualenv_modules": ["mercurial", "mozmill"]},
View
@@ -274,10 +274,8 @@ def run_peptest(self):
# TODO create a better summary for peptest
# for now just display return code
- self.add_summary("%s exited with return code %s: %s" % (cmd[0],
- code,
- status),
- level=level)
+ self.log("%s exited with return code %s: %s" % (cmd[0], code, status),
+ level=level)
self.buildbot_status(tbpl_status)
@@ -136,6 +136,7 @@ def __init__(self, require_config_file=True):
"sign",
"verify-signatures",
"upload-signed-bits",
+ "summary",
],
require_config_file=require_config_file
)
@@ -202,7 +202,7 @@ def source(self):
self.fatal("I don't know how to handle vcs '%s'!" % self.config['vcs'])
got_revision = vcs_obj.ensure_repo_and_revision()
- self.add_summary("Got revision %s\n" % got_revision)
+ self.info("Got revision %s\n" % got_revision)
# __main__ {{{1
if __name__ == '__main__':

0 comments on commit 0965962

Please sign in to comment.