Merge ~kissiel/checkbox-provider-phoronix:stricter-approach into checkbox-provider-phoronix:master

Proposed by Maciej Kisielewski
Status: Merged
Approved by: Maciej Kisielewski
Approved revision: 6475f0719508b1fe39d43c6c152e08b39b5178af
Merged at revision: 4fd71d10208def571219962711697df15076b5ce
Proposed branch: ~kissiel/checkbox-provider-phoronix:stricter-approach
Merge into: checkbox-provider-phoronix:master
Diff against target: 73 lines (+12/-7)
3 files modified
requirements/container-tests-provider-phoronix (+3/-0)
units/packaging.pxu (+1/-1)
units/pts.pxu (+8/-6)
Reviewer Review Type Date Requested Status
Devices Certification Bot Needs Fixing
Sylvain Pineau (community) Approve
Paul Larson Approve
Review via email: mp+380104@code.launchpad.net

Description of the change

* make PTS require the cache, so it never installs the tests
* don't run on anything else than x86_64 (for now)

To post a comment you must log in.
Revision history for this message
Paul Larson (pwlars) wrote :

lgtm, +1

review: Approve
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

+1

review: Approve
Revision history for this message
Devices Certification Bot (ce-certification-qa) wrote :

The merge was fine but running tests failed.

[xenial] [17:11:14] starting container
Device project added to xenial-testing
[xenial] [17:11:24] provisioning container
[xenial] [17:12:07] Starting tests...
[xenial] Found a test script: ./requirements/container-tests-provider-phoronix
[xenial] [17:12:37] container-tests-provider-phoronix: FAIL
[xenial] output: https://paste.ubuntu.com/p/RBHmdgcHgy/
[xenial] [17:12:40] Fixing file permissions in source directory
[xenial] [17:12:40] Destroying container
[bionic] [17:12:57] starting container
Device project added to bionic-testing
[bionic] [17:13:10] provisioning container
[bionic] [17:13:51] Starting tests...
[bionic] Found a test script: ./requirements/container-tests-provider-phoronix
[bionic] [17:14:22] container-tests-provider-phoronix: FAIL
[bionic] output: https://paste.ubuntu.com/p/kNjWgzyPyz/
[bionic] [17:14:24] Fixing file permissions in source directory
[bionic] [17:14:25] Destroying container

review: Needs Fixing

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/requirements/container-tests-provider-phoronix b/requirements/container-tests-provider-phoronix
2index 5fc1891..311b403 100755
3--- a/requirements/container-tests-provider-phoronix
4+++ b/requirements/container-tests-provider-phoronix
5@@ -9,4 +9,7 @@ git clone git://git.launchpad.net/checkbox-ng $TMPDIR/checkbox-ng
6 echo "I: running 'develop' on checkbox-ng"
7 ( cd $TMPDIR/checkbox-ng/ && python3 setup.py develop --quiet | sed -e 's/^/I (develop output) /' )
8
9+git clone git://git.launchpad.net/plainbox-provider-resource $TMPDIR/plainbox-provider-resource
10+python3 $TMPDIR/plainbox-provider-resource/manage.py develop --force
11+
12 ./manage.py validate
13diff --git a/units/packaging.pxu b/units/packaging.pxu
14index bd8f50a..f45baff 100644
15--- a/units/packaging.pxu
16+++ b/units/packaging.pxu
17@@ -1,3 +1,3 @@
18 unit: packaging meta-data
19 os-id: debian
20-Depends: php-cli, php-xml, xserver-xorg-dev
21+Depends: php-cli, php-xml, xserver-xorg-dev, plainbox-provider-resource-generic
22diff --git a/units/pts.pxu b/units/pts.pxu
23index 9b773ff..cadfe36 100644
24--- a/units/pts.pxu
25+++ b/units/pts.pxu
26@@ -12,11 +12,13 @@ include:
27 pts-generate-report
28 pts-attach-report
29 bootstrap_include:
30+ com.canonical.certification::cpuinfo
31 pts-job-generator
32
33 id: pts-job-generator
34 category_id: phoronix
35 plugin: resource
36+requires: cpuinfo.platform == "x86_64"
37 command:
38 echo id: pts/cpuminer-opt
39 echo "name: Cpuminer benchmark."
40@@ -42,10 +44,12 @@ category_id: phoronix
41 plugin: shell
42 command:
43 mkdir $PLAINBOX_SESSION_SHARE/pts-nest/
44- if [ ! -z "$PTS_PRELOAD_PATH" ]; then
45- cp -r $PTS_PRELOAD_PATH/* $PLAINBOX_SESSION_SHARE/pts-nest/
46- rm -rf $PLAINBOX_SESSION_SHARE/pts-nest/test-results/*
47+ if [ -z "$PTS_PRELOAD_PATH" ]; then
48+ echo "PTS_PRELOAD_PATH is not defined! PTS tests cannot run" >&2
49+ exit 1
50 fi
51+ cp -r $PTS_PRELOAD_PATH/* $PLAINBOX_SESSION_SHARE/pts-nest/
52+ rm -rf $PLAINBOX_SESSION_SHARE/pts-nest/test-results/*
53 cp $PLAINBOX_PROVIDER_DATA/phoronix-test-suite-configuration/* $PLAINBOX_SESSION_SHARE/pts-nest
54 sed -i "s+__TARGET_PATH__+$PLAINBOX_SESSION_SHARE\/pts-nest\/+" $PLAINBOX_SESSION_SHARE/pts-nest/user-config.xml
55 environ: PTS_PRELOAD_PATH
56@@ -57,9 +61,6 @@ category_id: phoronix
57 _summary: {name}
58 command:
59 export PTS_USER_PATH_OVERRIDE=$PLAINBOX_SESSION_SHARE/pts-nest/
60- if [ -z "$PTS_PRELOAD_PATH" ]; then
61- $PLAINBOX_PROVIDER_DATA/phoronix-test-suite/phoronix-test-suite install {id}
62- fi
63 pts_wrapper.py {id}
64 plugin: shell
65 depends: pts-prepare-nest
66@@ -69,6 +70,7 @@ environ: PTS_PRELOAD_PATH
67 id: pts-generate-report
68 category_id: phoronix
69 plugin: shell
70+depends: pts-prepare-nest
71 command:
72 export PTS_USER_PATH_OVERRIDE=$PLAINBOX_SESSION_SHARE/pts-nest/
73 cd $PLAINBOX_SESSION_SHARE/pts-nest/test-results/

Subscribers

People subscribed via source and target branches