fix(runner): 0 max parallel tasks mean infity

semaphoreui · Oct 15, 2024 · 4016c85 · 4016c85
1 parent 2f7c25b
commit 4016c85
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/services/tasks/RemoteJob.go b/services/tasks/RemoteJob.go
@@ -95,7 +95,7 @@ func (t *RemoteJob) Run(username string, incomingVersion *string) (err error) {
 
 	for _, r := range runners {
 		n := t.taskPool.GetNumberOfRunningTasksOfRunner(r.ID)
-		if n < r.MaxParallelTasks {
+		if n > 0 && n < r.MaxParallelTasks {
 			runner = &r
 			break
 		}

diff --git a/web/src/components/RunnerForm.vue b/web/src/components/RunnerForm.vue
@@ -53,6 +53,12 @@ export default {
       return '/api/runners';
     },
 
+    beforeSave() {
+      if (!this.item.max_parallel_tasks) {
+        this.item.max_parallel_tasks = 0;
+      }
+    },
+
     getSingleItemUrl() {
       return `/api/runners/${this.itemId}`;
     },