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

Unified Diff: infra/bots/recipes/swarm_perf.py

Issue 2366623003: Include timestamp in nanobench JSON file name (Closed)
Patch Set: Also, don't download the nanobench JSON Created 4 years, 3 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 | « no previous file | infra/bots/recipes/swarm_perf.expected/Perf-Android-Clang-Nexus6-GPU-Adreno420-arm-Release-GN_Android.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: infra/bots/recipes/swarm_perf.py
diff --git a/infra/bots/recipes/swarm_perf.py b/infra/bots/recipes/swarm_perf.py
index d9927eac74ce3144318592b710233e81bfb6c7f0..fb8838c047f227a86730b2f43e0dc4b913a31d83 100644
--- a/infra/bots/recipes/swarm_perf.py
+++ b/infra/bots/recipes/swarm_perf.py
@@ -14,6 +14,7 @@ DEPS = [
'recipe_engine/platform',
'recipe_engine/properties',
'recipe_engine/raw_io',
+ 'recipe_engine/time',
'run',
'flavor',
'vars',
@@ -43,6 +44,9 @@ TEST_BUILDERS = {
}
+import time
+
+
def nanobench_flags(bot):
args = ['--pre_log']
@@ -201,9 +205,11 @@ def perf_steps(api):
args.extend(nanobench_flags(api.vars.builder_name))
if api.vars.upload_perf_results:
+ now = api.time.utcnow()
+ ts = int(time.mktime(now.utctimetuple()))
json_path = api.flavor.device_path_join(
api.flavor.device_dirs.perf_data_dir,
- 'nanobench_%s.json' % api.vars.got_revision)
+ 'nanobench_%s_%d.json' % (api.vars.got_revision, ts))
args.extend(['--outResultsFile', json_path])
args.extend(properties)
« no previous file with comments | « no previous file | infra/bots/recipes/swarm_perf.expected/Perf-Android-Clang-Nexus6-GPU-Adreno420-arm-Release-GN_Android.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698