StrixOS/build
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Store metrics for each run of a benchmark." into main am: 0296ef0081

Browse Source 
Original change: https://android-review.googlesource.com/c/platform/build/+/2933374

Change-Id: I09dc713074a47e8d672af40238f431d92472f1e4
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Treehugger Robot
2024-01-31 19:16:57 +00:00
committed by Automerger Merge Worker
parent e063d96aeb 0296ef0081
commit 0d3c6b45d7
1 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
tools/perf/benchmarks
View File

@@ -368,6 +368,7 @@ class Runner():
# If we're disting just one benchmark, save the logs and we can stop here. # If we're disting just one benchmark, save the logs and we can stop here.
self._dist(utils.get_dist_dir(), benchmark.dumpvars) self._dist(utils.get_dist_dir(), benchmark.dumpvars)
else: else:
self._dist(benchmark_log_dir, benchmark.dumpvars, store_metrics_only=True)
# Postroll builds # Postroll builds
for i in range(benchmark.postroll): for i in range(benchmark.postroll):
ns = self._run_build(lunch, benchmark_log_dir.joinpath(f"post_{i}"), ns = self._run_build(lunch, benchmark_log_dir.joinpath(f"post_{i}"),
@@ -418,16 +419,19 @@ class Runner():
return after_ns - before_ns return after_ns - before_ns
def _dist(self, dist_dir, dumpvars): def _dist(self, dist_dir, dumpvars, store_metrics_only=False):
out_dir = utils.get_out_dir() out_dir = utils.get_out_dir()
dest_dir = dist_dir.joinpath("logs") dest_dir = dist_dir.joinpath("logs")
os.makedirs(dest_dir, exist_ok=True) os.makedirs(dest_dir, exist_ok=True)
basenames = [ basenames = [
"build.trace.gz",
"soong.log",
"soong_build_metrics.pb", "soong_build_metrics.pb",
"soong_metrics", "soong_metrics",
] ]
if not store_metrics_only:
basenames.extend([
"build.trace.gz",
"soong.log",
])
if dumpvars: if dumpvars:
basenames = ['dumpvars-'+b for b in basenames] basenames = ['dumpvars-'+b for b in basenames]
for base in basenames: for base in basenames: