Merge lp:~terceiro/lava-tool/remove-submit-parameters into lp:~linaro-validation/lava-tool/trunk

Proposed by Antonio Terceiro
Status: Merged
Approved by: Milo Casagrande
Approved revision: 195
Merged at revision: 195
Proposed branch: lp:~terceiro/lava-tool/remove-submit-parameters
Merge into: lp:~linaro-validation/lava-tool/trunk
Diff against target: 50 lines (+3/-18)
2 files modified
.gitignore (+3/-0)
lava/job/templates.py (+0/-18)
To merge this branch: bzr merge lp:~terceiro/lava-tool/remove-submit-parameters
Reviewer Review Type Date Requested Status
Milo Casagrande (community) Approve
Review via email: mp+182794@code.launchpad.net

Description of the change

job templates: remove server & bundle stream params

This makes it impossible to submit to a local device when you don't have
a full LAVA setup since it will always break at the "submit results"
job.

Also, there are plans to drop the server argument from submit results,
cfe https://bugs.launchpad.net/lava-dispatcher/+bug/1217061

I would even go further and make the results submission automatic, so
you shouldn't need an explicit submit-results action at all.

(you will note that I am also adding a .gitignore file since I am testing git-bzr as a first step migrating all our stuff into git)

To post a comment you must log in.
Revision history for this message
Milo Casagrande (milo) wrote :

Thanks Antonio for taking care of this, I was the plan to drop that parameter from the dispatcher.
And +10 on the .gitignore.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== added file '.gitignore'
--- .gitignore 1970-01-01 00:00:00 +0000
+++ .gitignore 2013-08-29 01:21:50 +0000
@@ -0,0 +1,3 @@
1*.pyc
2tags
3*.egg-info
04
=== modified file 'lava/job/templates.py'
--- lava/job/templates.py 2013-07-26 08:10:16 +0000
+++ lava/job/templates.py 2013-08-29 01:21:50 +0000
@@ -30,10 +30,6 @@
30TESTDEF_URLS_PARAMETER = ListParameter("testdef_urls")30TESTDEF_URLS_PARAMETER = ListParameter("testdef_urls")
31TESTDEF_URLS_PARAMETER.store = False31TESTDEF_URLS_PARAMETER.store = False
3232
33# Use another ID for the server parameter, might be different.
34SERVER_PARAMETER = Parameter("stream_server")
35STREAM_PARAMETER = Parameter("stream")
36
37BOOT_TEST = {33BOOT_TEST = {
38 "timeout": 18000,34 "timeout": 18000,
39 "job_name": "Boot test",35 "job_name": "Boot test",
@@ -68,13 +64,6 @@
68 "timeout": 1800,64 "timeout": 1800,
69 "testdef_urls": TESTDEF_URLS_PARAMETER,65 "testdef_urls": TESTDEF_URLS_PARAMETER,
70 }66 }
71 },
72 {
73 "command": "submit_results",
74 "parameters": {
75 "stream": STREAM_PARAMETER,
76 "server": SERVER_PARAMETER
77 }
78 }67 }
79 ]68 ]
80}69}
@@ -103,13 +92,6 @@
103 }92 }
104 ]93 ]
105 }94 }
106 },
107 {
108 "command": "submit_results",
109 "parameters": {
110 "stream": STREAM_PARAMETER,
111 "server": SERVER_PARAMETER
112 }
113 }95 }
114 ]96 ]
115}97}

Subscribers

People subscribed via source and target branches