Bug 9623

Summary:	Rewrite build-perf-test.sh in python
Product:	[QA/Testing] Build Testing	Reporter:	Markus Lehtonen <markus.lehtonen>
Component:	general	Assignee:	Markus Lehtonen <markus.lehtonen>
Status:	VERIFIED FIXED	QA Contact:	Jose Perez C <jose.perez.carranza>
Severity:	enhancement
Priority:	Medium+	CC:	benjamin.esquivel, humberto.ibarra.lopez, jose.perez.carranza, joshuagloe, richard.purdie, sgw
Version:	unspecified
Target Milestone:	2.2 M2
Hardware:	x86
OS:	Multiple
Whiteboard:
OS type for building Yocto:	---	Type of Regression:	---
Verified:		Documentation change:	No (bug/feature does not impact docs)

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