fix(runners): add atimic checks

config-runner.json

@@ -0,0 +1,13 @@
+{
+ 	"bolt": {
+ 		"host": "/Users/fiftin/go/src/github.com/ansible-semaphore/semaphore/database.boltdb"
+ 	},
+
+ 	"dialect": "bolt",
+ 	"tmp_path": "/var/folders/x1/d7p_yr4j7g57_r2r8s0ll_1r0000gn/T/semaphore",
+
+	"runner": {
+		"config_file": "/tmp/semaphore-runner.json",
+		"api_url": "http://localhost:3000/api"
+	}
+}

services/runners/JobPool.go

@@ -20,6 +20,7 @@ import (
 	"os"
 	"os/exec"
 	"strconv"
+	"sync/atomic"
 	"time"
 )
 
@@ -106,6 +107,9 @@ type JobPool struct {
 	queue []*job
 
 	config *RunnerConfig
+
+	checking int32
+	sending  int32
 }
 
 type RunnerRegistration struct {
@@ -256,6 +260,10 @@ func (p *JobPool) Run() {
 
 func (p *JobPool) sendProgress() {
 
+	if !atomic.CompareAndSwapInt32(&p.sending, 0, 1) {
+		return
+	}
+
 	client := &http.Client{}
 
 	url := util.Config.Runner.ApiURL + "/runners/" + strconv.Itoa(p.config.RunnerID)
@@ -289,6 +297,7 @@ func (p *JobPool) sendProgress() {
 	}
 
 	defer resp.Body.Close()
+	defer atomic.StoreInt32(&p.sending, 0)
 }
 
 func (p *JobPool) tryRegisterRunner() bool {
@@ -381,6 +390,12 @@ func (p *JobPool) tryRegisterRunner() bool {
 // checkNewJobs tries to find runner to queued jobs
 func (p *JobPool) checkNewJobs() {
 
+	if !atomic.CompareAndSwapInt32(&p.checking, 0, 1) {
+		return
+	}
+
+	defer atomic.StoreInt32(&p.checking, 0)
+
 	client := &http.Client{}
 
 	url := util.Config.Runner.ApiURL + "/runners/" + strconv.Itoa(p.config.RunnerID)

services/tasks/TaskRunner.go

@@ -64,7 +64,6 @@ func (t *TaskRunner) SetStatus(status db.TaskStatus) {
 	switch t.Task.Status { // check old status
 	case db.TaskRunningStatus:
 		if status == db.TaskWaitingStatus {
-			//panic("running TaskRunner cannot be " + status)
 			return
 		}
 		break
@@ -77,7 +76,6 @@ func (t *TaskRunner) SetStatus(status db.TaskStatus) {
 	case db.TaskSuccessStatus:
 	case db.TaskFailStatus:
 	case db.TaskStoppedStatus:
-		//panic("stopped TaskRunner cannot be " + status)
 		return
 	}