Merge lp://staging/~psivaa/ue-qa-kernel-tests-infrastructure/run-manual-omap4-tests into lp://staging/ue-qa-kernel-tests-infrastructure
Proposed by
Para Siva
| Status: | Needs review |
|---|---|
| Proposed branch: | lp://staging/~psivaa/ue-qa-kernel-tests-infrastructure/run-manual-omap4-tests |
| Merge into: | lp://staging/ue-qa-kernel-tests-infrastructure |
| Diff against target: |
208 lines (+204/-0) 1 file modified
kernel-sru/run_tests_manual_omap4.sh (+204/-0) |
| To merge this branch: | bzr merge lp://staging/~psivaa/ue-qa-kernel-tests-infrastructure/run-manual-omap4-tests |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Paul Larson | Needs Fixing | ||
|
Review via email:
|
|||
Description of the change
Added a script to run the kernel sru tests for pre- quantal releases on panda board. sshpass needs to be installed on the host system and also openssh server needs to be installed in the target panda board for this script to work.
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
A few minor things:
12 +# -r: The ubuntu release to be tested
Is this necessary? can't we just grab the release from the installed image since it has to be manually installed?
14 +# -c: autotest-
15 +# -a: autotest version from the git repo (v0.0.6-1)
16 +# -k: kernel-testing version from the git repo (v0.0.12.1)
I don't think we should specify all this. Nor do I think we should tar them up and push it across ssh. It's far easier to just install git on the test system, and git clone the appropriate repos right there on the panda.
83 +# Could use the available one in the branch but rewriting it
84 +# for using it standalone
85 +set_debconf_
86 +{
87 + local HOSTNAME=$1
88 + cat << EOF > /tmp/exim4.debconf
What's this for? and why do we need to write it here? because of the $HOSTNAME substitutions I guess?
Otherwise, I'm assuming you've tried this and it works. It wouldn't hurt to add a README or something saying what steps need to be performed before running it. This should make things easier when running those pesky manual-setup preinstalled image jobs. Thanks for working on this!