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: 276
Proposed branch: lp:~stylesen/lava-scheduler/multinode
Merge into: lp:lava-scheduler/multinode
Diff against target: 37 lines (+14/-2)
2 files modified
lava_scheduler_app/api.py (+7/-1)
lava_scheduler_app/views.py (+7/-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+180509@code.launchpad.net

Description of the change

Add ability to cancel multinode jobs.

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

Approved, thanks.

review: Approve
276. By Neil Williams

Senthil Kumaran 2013-08-16 Add ability to cancel multinode jobs.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lava_scheduler_app/api.py'
--- lava_scheduler_app/api.py 2013-08-16 07:59:57 +0000
+++ lava_scheduler_app/api.py 2013-08-16 09:30:49 +0000
@@ -60,7 +60,13 @@
60 job = TestJob.objects.get(pk=job_id)60 job = TestJob.objects.get(pk=job_id)
61 if not job.can_cancel(self.user):61 if not job.can_cancel(self.user):
62 raise xmlrpclib.Fault(403, "Permission denied.")62 raise xmlrpclib.Fault(403, "Permission denied.")
63 job.cancel()63 if job.target_group:
64 multinode_jobs = TestJob.objects.all().filter(
65 target_group=job.target_group)
66 for multinode_job in multinode_jobs:
67 multinode_job.cancel()
68 else:
69 job.cancel()
64 return True70 return True
6571
66 def job_output(self, job_id):72 def job_output(self, job_id):
6773
=== modified file 'lava_scheduler_app/views.py'
--- lava_scheduler_app/views.py 2013-08-16 07:59:57 +0000
+++ lava_scheduler_app/views.py 2013-08-16 09:30:49 +0000
@@ -800,7 +800,13 @@
800def job_cancel(request, pk):800def job_cancel(request, pk):
801 job = get_restricted_job(request.user, pk)801 job = get_restricted_job(request.user, pk)
802 if job.can_cancel(request.user):802 if job.can_cancel(request.user):
803 job.cancel()803 if job.target_group:
804 multinode_jobs = TestJob.objects.all().filter(
805 target_group=job.target_group)
806 for multinode_job in multinode_jobs:
807 multinode_job.cancel()
808 else:
809 job.cancel()
804 return redirect(job)810 return redirect(job)
805 else:811 else:
806 return HttpResponseForbidden(812 return HttpResponseForbidden(

Subscribers

People subscribed via source and target branches

to status/vote changes: