Merge lp:~le-chi-thu/linaro-ci/add-testplan-support into lp:linaro-ci
Proposed by
Le Chi Thu
Status: | Merged |
---|---|
Merged at revision: | 62 |
Proposed branch: | lp:~le-chi-thu/linaro-ci/add-testplan-support |
Merge into: | lp:linaro-ci |
Diff against target: |
129 lines (+43/-34) 1 file modified
get_latest_ci_hwpack (+43/-34) |
To merge this branch: | bzr merge lp:~le-chi-thu/linaro-ci/add-testplan-support |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Deepti B. Kalakeri (community) | Approve | ||
Le Chi Thu (community) | Needs Resubmitting | ||
Review via email: mp+104743@code.launchpad.net |
Description of the change
Add support to define test plan for lava.
the format is : LAVA_TEST_
options and timeout is optional.
Examples:
LAVA_TEST_
LAVA_TEST_
if the LAVA_TEST_PLAN is not define, the default will be "ltp,pwrmgmt". It mean the current jobs do not need to update.
To post a comment you must log in.
Thanks for the changes. Please see my comments inline.
On Fri, May 4, 2012 at 7:38 PM, Le Chi Thu <email address hidden> wrote:
> Le Chi Thu has proposed merging /code.launchpad .net/~le- chi-thu/ linaro- ci/add- testplan- support/ +merge/ 104743<https:/ /code.launchpad .net/%7Ele- chi-thu/ linaro- ci/add- testplan- support/ +merge/ 104743> PLAN=<testcase> (<options> ):<timeout> ,... PLAN=ltp, pwrmgmt PLAN=test_ abc(option_ abc):100
> lp:~le-chi-thu/linaro-ci/add-testplan-support into lp:linaro-ci.
>
> Requested reviews:
> Deepti B. Kalakeri (deeptik)
>
> For more details, see:
>
> https:/
>
> Add support to define test plan for lava.
>
> the format is : LAVA_TEST_
> options and timeout is optional.
>
> Examples:
> LAVA_TEST_
> LAVA_TEST_
>
Could you give me an example of the options_abc ?
>
> if the LAVA_TEST_PLAN is not define, the default will be "ltp,pwrmgmt". It
> mean the current jobs do not need to update.
>
The default test that needs to be run should as well include
boot_linaro_image. Without booting the image how will you be able to run
the tests ?
> -- /code.launchpad .net/~le- chi-thu/ linaro- ci/add- testplan- support/ +merge/ 104743<https:/ /code.launchpad .net/%7Ele- chi-thu/ linaro- ci/add- testplan- support/ +merge/ 104743> ci_hwpack' ci_hwpack 2012-04-04 12:14:43 +0000 ci_hwpack 2012-05-04 14:07:20 +0000 "KERNEL_ JOB_NAME" , "unknown") + '_' + \ "HWPACK_ BUILD_DATE" , "unknown") + '_daily test' "LAVA_TEST_ PLAN", "ltp,pwrmgmt") linaro_ image" install" ,
>
> https:/
> You are requested to review the proposed merge of
> lp:~le-chi-thu/linaro-ci/add-testplan-support into lp:linaro-ci.
>
> === modified file 'get_latest_
> --- get_latest_
> +++ get_latest_
> @@ -5,6 +5,7 @@
>
> Usage: ./gentemplate <target> <hwpack_type> <image_type>
> """
> +import re
>
> from find_latest import find_latest_hwpack, find_latest_rootfs,
> find_ci_latest
> import json
> @@ -15,8 +16,7 @@
> job_name = os.getenv(
> os.getenv(
>
> -
> -
> +testplan = os.getenv(
>
> template = {
> "timeout": 20000,
> @@ -25,36 +25,6 @@
> "command": "deploy_
> },
> {
> - "command": "lava_test_
> - "parameters":
> - {
> - "tests": [
> - "ltp",
> - "pwrmgmt"
> - ],
> - "timeout": 4000
> - }
> - },
> - {
> - "command": "boot_linaro_image"
> - },
>
This should not be deleted, as we need to boot the image and then can we
run the tests.
> - { "BUNDLE_ STREAM_ NAME", "/anonymous/ plars/" ), dir_url= os.getenv( "KERNEL_ JOB_NAME" , "unknown") join(hwpack_ url_path, hwpack_dir_url)
> - "command": "lava_test_run",
> - "parameters":
> - {
> - "test_name": "ltp",
> - "timeout": 4800
> - }
> - },
> - {
> - "command": "lava_test_run",
> - "parameters":
> - {
> - "test_name": "pwrmgmt",
> - "timeout": 4800
> - }
> - },
> - {
> "command": "submit_results",
> "parameters": {
> "stream": os.getenv(
> @@ -71,6 +41,9 @@
> hwpack_
> hwpack_url_path = os.path.join(url, hwpack_parent_url)
> hwpack_url_path = os.path.
> +
> +
> +
>
Please delete the blank lines, you can leave a single bla...