Merge lp:~stylesen/lava-scheduler/multinode into lp:lava-scheduler/multinode

Proposed by Senthil Kumaran S
Status: Merged
Approved by: Neil Williams
Approved revision: no longer in the source branch.
Merged at revision: 263
Proposed branch: lp:~stylesen/lava-scheduler/multinode
Merge into: lp:lava-scheduler/multinode
Diff against target: 36 lines (+14/-1)
2 files modified
lava_scheduler_app/templates/lava_scheduler_app/job_submit.html (+10/-0)
lava_scheduler_app/views.py (+4/-1)
To merge this branch: bzr merge lp:~stylesen/lava-scheduler/multinode
Reviewer Review Type Date Requested Status
Neil Williams Approve
Review via email: mp+172974@code.launchpad.net

Description of the change

Fix error while submitting multinode job via UI.

To post a comment you must log in.
Revision history for this message
Neil Williams (codehelp) wrote :

Tested on multinode.validation.linaro.org, approved.

review: Approve
263. By Neil Williams

Senthil Kumaran 2013-07-04 Fix error while submitting multinode job via UI.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lava_scheduler_app/templates/lava_scheduler_app/job_submit.html'
--- lava_scheduler_app/templates/lava_scheduler_app/job_submit.html 2013-06-28 09:43:44 +0000
+++ lava_scheduler_app/templates/lava_scheduler_app/job_submit.html 2013-07-04 08:56:28 +0000
@@ -31,6 +31,16 @@
31To view the full job list click <a href="{{ list_url }}">here</a>.31To view the full job list click <a href="{{ list_url }}">here</a>.
32</div>32</div>
3333
34{% elif job_list %}
35{% url lava.scheduler.job.list as list_url %}
36<div id="job-success">Multinode Job submission successfull!
37<br>
38<br>
39Jobs with ID {{ job_list }}</a> has been created.
40<br>
41To view the full job list click <a href="{{ list_url }}">here</a>.
42</div>
43
34{% else %}44{% else %}
3545
36{% if error %}46{% if error %}
3747
=== modified file 'lava_scheduler_app/views.py'
--- lava_scheduler_app/views.py 2013-07-04 05:22:29 +0000
+++ lava_scheduler_app/views.py 2013-07-04 08:56:28 +0000
@@ -599,7 +599,10 @@
599 job = TestJob.from_json_and_user(599 job = TestJob.from_json_and_user(
600 request.POST.get("json-input"), request.user)600 request.POST.get("json-input"), request.user)
601601
602 response_data["job_id"] = job.id602 if isinstance(job, type(list())):
603 response_data["job_list"] = job
604 else:
605 response_data["job_id"] = job.id
603 return render_to_response(606 return render_to_response(
604 "lava_scheduler_app/job_submit.html",607 "lava_scheduler_app/job_submit.html",
605 response_data, RequestContext(request))608 response_data, RequestContext(request))

Subscribers

People subscribed via source and target branches

to status/vote changes: