Merge lp:~cypressyew/checkbox-certification/whitelist-location-change into lp:checkbox-certification

Proposed by Po-Hsu Lin
Status: Merged
Approved by: Zygmunt Krynicki
Approved revision: 645
Merged at revision: 645
Proposed branch: lp:~cypressyew/checkbox-certification/whitelist-location-change
Merge into: lp:checkbox-certification
Diff against target: 75 lines (+10/-5)
5 files modified
bin/checkbox-certification-client (+1/-1)
bin/checkbox-certification-prepare (+2/-2)
bin/checkbox-certification-server (+1/-1)
bin/checkbox-certification-server-soc (+1/-1)
debian/changelog (+5/-0)
To merge this branch: bzr merge lp:~cypressyew/checkbox-certification/whitelist-location-change
Reviewer Review Type Date Requested Status
Zygmunt Krynicki (community) Approve
Review via email: mp+204828@code.launchpad.net

Description of the change

Since we're moving whitelists out of checkbox-certificaton, corresponding changes must be made.
Affecting:
bin/checkbox-certification-prepare,
bin/checkbox-certification-server-soc
bin/checkbox-certification-server
bin/checkbox-certification-client

Note that the whitelist location is slightly different in Checkbox.

To post a comment you must log in.
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

Looks good to me, thank you

+1

review: Approve
Revision history for this message
Po-Hsu Lin (cypressyew) wrote :

Aaah, I didn't notice that Brendan already proposed a branch for this...

Revision history for this message
Brendan Donegan (brendan-donegan) wrote :

That's ok, I'll delete the other merge request

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'bin/checkbox-certification-client'
--- bin/checkbox-certification-client 2012-06-05 16:39:06 +0000
+++ bin/checkbox-certification-client 2014-02-05 07:23:40 +0000
@@ -8,7 +8,7 @@
8export CHECKBOX_DATA=${CHECKBOX_DATA:-~/.checkbox}8export CHECKBOX_DATA=${CHECKBOX_DATA:-~/.checkbox}
9export CHECKBOX_SHARE=${CHECKBOX_SHARE:-/usr/share/checkbox}9export CHECKBOX_SHARE=${CHECKBOX_SHARE:-/usr/share/checkbox}
10export CHECKBOX_CERTIFICATION_SHARE=${CHECKBOX_CERTIFICATION_SHARE:-.}10export CHECKBOX_CERTIFICATION_SHARE=${CHECKBOX_CERTIFICATION_SHARE:-.}
11export CHECKBOX_OPTIONS=${CHECKBOX_OPTIONS:---log=$CHECKBOX_DATA/checkbox-certification.log --whitelist-file=$CHECKBOX_CERTIFICATION_SHARE/data/client-selftest.whitelist $OPTIONS}11export CHECKBOX_OPTIONS=${CHECKBOX_OPTIONS:---log=$CHECKBOX_DATA/checkbox-certification.log --whitelist-file=$CHECKBOX_SHARE/data/whitelists/client-selftest.whitelist $OPTIONS}
1212
13# Convenience for defining the PYTHONPATH directory.13# Convenience for defining the PYTHONPATH directory.
14if [ "$CHECKBOX_SHARE" != "/usr/share/checkbox" ]; then14if [ "$CHECKBOX_SHARE" != "/usr/share/checkbox" ]; then
1515
=== modified file 'bin/checkbox-certification-prepare'
--- bin/checkbox-certification-prepare 2013-11-15 03:33:08 +0000
+++ bin/checkbox-certification-prepare 2014-02-05 07:23:40 +0000
@@ -29,7 +29,7 @@
29cat >~/run-checkbox.sh <<EOF29cat >~/run-checkbox.sh <<EOF
30checkbox-certification-client \30checkbox-certification-client \
31--config="checkbox-certification/plugins/submission_info/tags=$TAGS" \31--config="checkbox-certification/plugins/submission_info/tags=$TAGS" \
32-W /usr/share/checkbox-certification/data/client-cert.whitelist \32-W /usr/share/checkbox/data/whitelists/client-cert.whitelist \
33--config="checkbox/plugins/launchpad_prompt/email=hardware-certification-devel@canonical.com" \33--config="checkbox/plugins/launchpad_prompt/email=hardware-certification-devel@canonical.com" \
34--config="checkbox/plugins/blacklist=" \34--config="checkbox/plugins/blacklist=" \
35--log-level=debug35--log-level=debug
@@ -55,7 +55,7 @@
55cat << EOF2 > ~/actually-run-checkbox-stress.sh55cat << EOF2 > ~/actually-run-checkbox-stress.sh
56checkbox-certification-client \56checkbox-certification-client \
57 --config=\""checkbox-certification/plugins/submission_info/tags=$TAGS\"" \57 --config=\""checkbox-certification/plugins/submission_info/tags=$TAGS\"" \
58 -W /usr/share/checkbox-certification/data/client-stress.whitelist \58 -W /usr/share/checkbox/data/whitelists/client-stress.whitelist \
59 --config="checkbox/plugins/launchpad_prompt/email=hardware-certification-devel@canonical.com" \59 --config="checkbox/plugins/launchpad_prompt/email=hardware-certification-devel@canonical.com" \
60 --config="checkbox/plugins/blacklist=" \60 --config="checkbox/plugins/blacklist=" \
61 --config="checkbox-certification/plugins/submission_info/secure_id=$submission_id"61 --config="checkbox-certification/plugins/submission_info/secure_id=$submission_id"
6262
=== modified file 'bin/checkbox-certification-server'
--- bin/checkbox-certification-server 2012-06-05 16:39:06 +0000
+++ bin/checkbox-certification-server 2014-02-05 07:23:40 +0000
@@ -8,7 +8,7 @@
8export CHECKBOX_DATA=${CHECKBOX_DATA:-~/.checkbox}8export CHECKBOX_DATA=${CHECKBOX_DATA:-~/.checkbox}
9export CHECKBOX_SHARE=${CHECKBOX_SHARE:-/usr/share/checkbox}9export CHECKBOX_SHARE=${CHECKBOX_SHARE:-/usr/share/checkbox}
10export CHECKBOX_CERTIFICATION_SHARE=${CHECKBOX_CERTIFICATION_SHARE:-.}10export CHECKBOX_CERTIFICATION_SHARE=${CHECKBOX_CERTIFICATION_SHARE:-.}
11export CHECKBOX_OPTIONS=${CHECKBOX_OPTIONS:---log=$CHECKBOX_DATA/checkbox-certification.log --whitelist-file=$CHECKBOX_CERTIFICATION_SHARE/data/server-selftest.whitelist $OPTIONS}11export CHECKBOX_OPTIONS=${CHECKBOX_OPTIONS:---log=$CHECKBOX_DATA/checkbox-certification.log --whitelist-file=$CHECKBOX_SHARE/data/whitelists/server-selftest.whitelist $OPTIONS}
1212
13# Convenience for defining the PYTHONPATH directory.13# Convenience for defining the PYTHONPATH directory.
14if [ "$CHECKBOX_SHARE" != "/usr/share/checkbox" ]; then14if [ "$CHECKBOX_SHARE" != "/usr/share/checkbox" ]; then
1515
=== modified file 'bin/checkbox-certification-server-soc'
--- bin/checkbox-certification-server-soc 2013-10-04 11:13:51 +0000
+++ bin/checkbox-certification-server-soc 2014-02-05 07:23:40 +0000
@@ -8,7 +8,7 @@
8export CHECKBOX_DATA=${CHECKBOX_DATA:-~/.checkbox}8export CHECKBOX_DATA=${CHECKBOX_DATA:-~/.checkbox}
9export CHECKBOX_SHARE=${CHECKBOX_SHARE:-/usr/share/checkbox}9export CHECKBOX_SHARE=${CHECKBOX_SHARE:-/usr/share/checkbox}
10export CHECKBOX_CERTIFICATION_SHARE=${CHECKBOX_CERTIFICATION_SHARE:-.}10export CHECKBOX_CERTIFICATION_SHARE=${CHECKBOX_CERTIFICATION_SHARE:-.}
11export CHECKBOX_OPTIONS=${CHECKBOX_OPTIONS:---log=$CHECKBOX_DATA/checkbox-certification.log --whitelist-file=$CHECKBOX_CERTIFICATION_SHARE/data/server-soc-selftest.whitelist $OPTIONS}11export CHECKBOX_OPTIONS=${CHECKBOX_OPTIONS:---log=$CHECKBOX_DATA/checkbox-certification.log --whitelist-file=$CHECKBOX_SHARE/data/whitelists/server-soc-selftest.whitelist $OPTIONS}
1212
13# Convenience for defining the PYTHONPATH directory.13# Convenience for defining the PYTHONPATH directory.
14if [ "$CHECKBOX_SHARE" != "/usr/share/checkbox" ]; then14if [ "$CHECKBOX_SHARE" != "/usr/share/checkbox" ]; then
1515
=== modified file 'debian/changelog'
--- debian/changelog 2014-02-03 14:27:24 +0000
+++ debian/changelog 2014-02-05 07:23:40 +0000
@@ -1,5 +1,10 @@
1checkbox-certification (0.19) UNRELEASED; urgency=low1checkbox-certification (0.19) UNRELEASED; urgency=low
22
3 [ Po-Hsu Lin ]
4 * Change the whitelist location of checkbox-certification-prepare,
5 checkbox-certification-server-soc, checkbox-certification-server
6 and checkbox-certification-client to Checkbox.
7
3 [ Brendan Donegan ]8 [ Brendan Donegan ]
49
5 * Changed python3 to python3-all in Build-Depends. 10 * Changed python3 to python3-all in Build-Depends.

Subscribers

People subscribed via source and target branches