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.pur...@linuxfoundation.org>
---
scripts/oe-build-perf-report | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
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
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#211102):
https://lists.openembedded.org/g/openembedded-core/message/211102
Mute This Topic: https://lists.openembedded.org/mt/111104827/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
