diff mbox series

[2/3] oe-build-perf-report: Use commit_time if available

Message ID 20250210161406.384031-2-richard.purdie@linuxfoundation.org
State Accepted, archived
Commit 8c22efd444b0b1486c43f01b41cba891a47c7520
Headers show
Series [1/3] libsecret: Inherit bash-completion | expand

Commit Message

Richard Purdie Feb. 10, 2025, 4:14 p.m. UTC 
Currently, the perf report charts use start_time which is when the build
is run. If running builds for commits in the past, this is inaccurate so
switch to using the recently added commit_time if the field is present.
Over time, the reports will switch to using the more accurate values.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
 scripts/oe-build-perf-report | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/scripts/oe-build-perf-report b/scripts/oe-build-perf-report
index 6c3c726ee3a..a36f3c1bca3 100755
--- a/scripts/oe-build-perf-report
+++ b/scripts/oe-build-perf-report
@@ -338,7 +338,11 @@  def print_html_report(data, id_comp, buildstats):
                 commit_num = get_data_item(meta, 'layers.meta.commit_count')
                 commit = get_data_item(meta, 'layers.meta.commit')
                 # Add start_time for both test measurement types of sysres and disk usage
-                start_time = test_i['start_time'][0]
+                try:
+                    # Use the commit_time if available, falling back to start_time
+                    start_time = get_data_item(meta, 'layers.meta.commit_time')
+                except KeyError:
+                    start_time = test_i['start_time'][0]
                 samples.append(measurement_stats(meas_i, '', start_time))
                 samples[-1]['commit_num'] = commit_num
                 samples[-1]['commit'] = commit