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

Unified Diff: scripts/slave/runtest.py

Issue 10826217: Script to archive profiling data collected from (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build/
Patch Set: Created 8 years, 4 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
« no previous file with comments | « scripts/slave/archive_profiling_data.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/runtest.py
===================================================================
--- scripts/slave/runtest.py (revision 149909)
+++ scripts/slave/runtest.py (working copy)
@@ -397,6 +397,31 @@
return build_dir, exe_path
+def upload_profiling_data(options):
+ """Using the target build configuration, archive the profiling data to Google
+ Storage.
+ """
+ build_dir = os.path.normpath(os.path.abspath(options.build_dir))
+
+ # archive_profiling_data.py is in
+ # /b/build/slave/SLAVE_NAME/build/src/build/scripts/slave
+ profiling_archive_tool = os.path.join(build_dir, 'src', 'build', 'scripts',
+ 'slave', 'archive_profiling_data.py')
+
+ if sys.platform == 'win32':
+ python = 'python_slave'
+ else:
+ python = 'python'
+
+ run_id = '%s_%s_%s' % (options.build_properties['got_revision'],
+ options.build_properties['buildername'],
cmp 2012/08/09 02:13:04 and these should be .get(), too :)
+ options.gtest_filter[0:12])
+ cmd = [python, profiling_archive_tool, '--run-id', run_id,
+ '--build-dir', build_dir]
+
+ return chromium_utils.RunCommand(cmd)
+
+
def main_mac(options, args):
if len(args) < 1:
raise chromium_utils.MissingArgument('Usage: %s' % USAGE)
@@ -899,6 +924,11 @@
sys.stderr.write('Unknown sys.platform value %s\n' % repr(sys.platform))
return 1
+ if (options.build_properties['buildername'] == 'XP Perf (1)' and
cmp 2012/08/09 02:13:04 right! this should be .get('buildername')
+ options.build_properties.get('mastername') == 'chromium.perf' and
+ options.gtest_filter == 'StartupTest.*:ShutdownTest.*'):
+ upload_profiling_data(options)
+
new_temp_files = get_temp_count()
if temp_files > new_temp_files:
print >> sys.stderr, (
« no previous file with comments | « scripts/slave/archive_profiling_data.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698