Skip to content

Commit

Permalink
fix(hooks): put the right status for outgoing hook
Browse files Browse the repository at this point in the history
Signed-off-by: Benjamin Coenen <5719034+bnjjj@users.noreply.github.com>
  • Loading branch information
bnjjj authored and richardlt committed Jan 17, 2020
1 parent a423ce2 commit b5de1c1
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion engine/hooks/outgoing_hooks.go
Original file line number Diff line number Diff line change
Expand Up @@ -235,7 +235,7 @@ func (s *Service) doOutgoingWorkflowExecution(ctx context.Context, t *sdk.TaskEx
}

callbackData.Log = fmt.Sprintf("Workflow %s/%s #%d.%d has been started", targetProject, targetWorkflow, targetRun.Number, targetRun.LastSubNumber)
callbackData.Status = sdk.StatusDisabled
callbackData.Status = sdk.StatusBuilding
callbackData.WorkflowRunNumber = &targetRun.Number

// Post the callback
Expand Down
2 changes: 1 addition & 1 deletion engine/worker/internal/take.go
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ func (w *CurrentWorker) Take(ctx context.Context, job sdk.WorkflowNodeJobRun) er
log.Info(ctx, "takeWorkflowJob> Cannot send build result: HTTP %v - worker cancelled - giving up", lasterr)
return nil
}
log.Warning(ctx, "takeWorkflowJob> Cannot send build result: HTTP %v - try: %d - new try in 15s", lasterr, try)
log.Warning(ctx, "takeWorkflowJob> Cannot send build result for job id %d: HTTP %v - try: %d - new try in 15s", job.ID, lasterr, try)
time.Sleep(15 * time.Second)
}
log.Error(ctx, "takeWorkflowJob> Could not send built result 10 times, giving up. job: %d", job.ID)
Expand Down

0 comments on commit b5de1c1

Please sign in to comment.