Bug 9623 - Rewrite build-perf-test.sh in python
Summary: Rewrite build-perf-test.sh in python
Status: VERIFIED FIXED
Alias: None
Product: Build Testing
Classification: QA/Testing
Component: general (show other bugs)
Version: unspecified
Hardware: x86 Multiple
: Medium+ enhancement
Target Milestone: 2.2 M2
Assignee: Markus Lehtonen
QA Contact: Jose Perez C
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2016-05-12 08:29 UTC by Markus Lehtonen
Modified: 2016-08-12 16:20 UTC (History)
6 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Markus Lehtonen 2016-05-12 08:29:03 UTC
Rewrite the build performance test script (scripts/contrib/build-perf-test.sh) in Python. This would make it much more maintainable and easier to extend, and, make it possible to have shared functionality with other oeqa utilities.
Comment 1 Benjamin Esquivel 2016-05-26 19:24:43 UTC
Also, it should allow integrating to an automated tool that uses this script for obtaining performance measurements.
Comment 2 Richard Purdie 2016-07-12 16:27:21 UTC
Resolved in the patches up to http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=678a02e112bead7a5ef73c7e74ac2599a544de1f
Comment 3 Jose Perez C 2016-07-14 17:37:02 UTC
I saw that in patch [1] a new file named "build-perf-test-wrapper.sh" was created and the main purpose of the bug was to migrate the script to Python. Markus Lehtonen are you able to explain why the main purpose changed and another .sh file was created? 


1.-  http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=678a02e112bead7a5ef73c7e74ac2599a544de1f
Comment 4 Jose Perez C 2016-07-25 19:38:41 UTC
Currently there is failure executing wrapper bug 10019 opened to track this.
Comment 5 Markus Lehtonen 2016-08-01 14:27:41 UTC
Our automated tester machines have a shell script that is used to run build perf tests. This shell script isn't/wasn't maintained in any VCS. The new shell script (i.e. build-perf-test-wrapper.sh) was added as a substitute for this script: as a wrapper for running build perf tests with a single shell command. A shell script like this is needed because the actual tester script (i.e. scripts/oe-build-perf-test) needs to be run in an initialized build environment. The shell script takes care of checking out the correct git revision and running oe-init-build-env, for example.
Comment 6 Jose Perez C 2016-08-12 16:20:04 UTC
Wrapper works correctly