Merge pull request #442 from UnitedTraders/441-crash-after-new-role

Fix crash when new role added to requirements
semaphoreui · Feb 27, 2018 · 0e0bef5 · 0e0bef5
2 parents 7a43a58 + 10ade65
commit 0e0bef5
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/api/tasks/runner.go b/api/tasks/runner.go
@@ -98,6 +98,12 @@ func (t *task) prepareRun() {
 		return
 	}
 
+	if err := t.runGalaxy(); err != nil {
+		t.log("Running galaxy failed: " + err.Error())
+		t.fail()
+		return
+	}
+
 	// todo: write environment
 
 	if err := t.listPlaybookHosts(); err != nil {
@@ -154,12 +160,6 @@ func (t *task) run() {
 	t.log("Started: " + strconv.Itoa(t.task.ID))
 	t.log("Run task with template: " + t.template.Alias + "\n")
 
-	if err := t.runGalaxy(); err != nil {
-		t.log("Running galaxy failed: " + err.Error())
-		t.fail()
-		return
-	}
-
 	if err := t.runPlaybook(); err != nil {
 		t.log("Running playbook failed: " + err.Error())
 		t.fail()