Hello, On Wed, 6 Feb 2013 15:42:26 +0800 YongQin Liu <[email protected]> wrote:
> Hi, Paul > > I submitted my change here: > > https://code.launchpad.net/~liuyq0307/linaro-android-build-tools/lava-test-shell/+merge/146789 > > Could you please help to review that? > > And about the following mail, do you have any comments? I was a bit out of context when received that mail, but as Andy had suggestions, I skipped asking basic questions, and now seeing the patch, I gathered it all together. Yes, seems ok to me, and I merged your patch, thanks. > > Thanks, > Yongqin Liu > > On 3 February 2013 18:37, YongQin Liu <[email protected]> wrote: > > > Hi, Andy Doan & Paul Sokolovsky & Vishal Bhoj & All > > > > Now lava-test-shell is supported by LAVA, > > and android team is considering adding the support on android-build. > > > > I want to add following 3 interfaces on the android-build, > > > > *1. lava-test-shell-url(url= > > http://cbuild.validation.linaro.org:80/helpers/scheduler/lava/testdef/lava-pandaes-usbdrive/gcc-4.8~bzr121031~christophe-lyon-vect-cost-model > > timeout=59400)* > > where timeout is optional > > when submitted, the action in job will be something like this: > > > > { > > "command": "lava_test_shell", > > "parameters": { > > "testdef_urls": [" > > http://cbuild.validation.linaro.org:80/helpers/scheduler/lava/testdef/lava-pandaes-usbdrive/gcc-4.8~bzr121031~christophe-lyon-vect-cost-model > > "], > > "timeout": 59400 > > } > > } > > > > > > *2. > > lava-test-shell-git(repo=git://git.linaro.org/qa/test-definitions.git > > revision=master testdef=android/binder.yaml timeout=-1) * where > > revison/testdef/timeout is optional when submitted, the action in > > job will be something like this: { > > "command": "lava_test_shell", > > "parameters": { > > "testdef_repos": [ > > { > > "git-repo": "git:// > > git.linaro.org/qa/test-definitions.git", > > "revision": "master", > > "testdef": "android/binder.yaml" > > } > > ], > > "timeout": -1 > > } > > } > > *3. lava-test-shell-bzr(repo=lp:~terceiro/+junk/lava-dev > > revision=112 timeout=-1)* > > where revison/testdef/timeout is optional > > when submitted, the action in job will be something like this: > > { > > "command": "lava_test_shell", > > "parameters": { > > "testdef_repos": [ > > { > > "bzr-repo": > > "lp:~terceiro/+junk/lava-dev", > > "revision": "112", > > "testdef": > > "jobs/android-busybox.yaml" } > > ], > > "timeout": -1 > > } > > } > > > > > > > > By these 3 types interface, we can do most of the things > > lava-test-shell supported. > > Although we can't specify the array form with one interface one > > time, we can do the same things by specifying multiple times the > > interface of the same type. > > like > > *lava-test-shell-url(url=url1),**lava-test-shell-url(url=url2),* so > > I think it's enough for us to use on android-build. > > > > About this, how do you think about that? > > Please feel free to give your comments. Thanks in advance! > > > > -- > > Thanks, > > Yongqin Liu > > --------------------------------------------------------------- > > #mailing list > > [email protected] <[email protected]> > > http://lists.linaro.org/mailman/listinfo/linaro-android > > [email protected] <[email protected]> > > http://lists.linaro.org/pipermail/linaro-validation > > > > > -- Best Regards, Paul Linaro.org | Open source software for ARM SoCs Follow Linaro: http://www.facebook.com/pages/Linaro http://twitter.com/#!/linaroorg - http://www.linaro.org/linaro-blog _______________________________________________ linaro-validation mailing list [email protected] http://lists.linaro.org/mailman/listinfo/linaro-validation
