fix(api): avoid double book by the same hatchery (#6720)

yesnault · web-flow · commit d66aad6ae05f · 2023-12-07T10:13:42.000+01:00
Signed-off-by: Yvonnick Esnault &lt;yvonnick.esnault@corp.ovh.com&gt;
diff --git a/engine/api/workflow/execute_node_job_run.go b/engine/api/workflow/execute_node_job_run.go
@@ -375,9 +375,6 @@ func BookNodeJobRun(ctx context.Context, store cache.Store, defaultBookDelay int
 		}
 		return nil, nil
 	}
-	if h.ID == hatchery.ID {
-		return nil, nil
-	}
 	return &h, sdk.WrapError(sdk.ErrJobAlreadyBooked, "BookNodeJobRun> job %d already booked by %s (%d)", id, h.Name, h.ID)
 }
 

Original file line number	Diff line number	Diff line change
`@@ -375,9 +375,6 @@ func BookNodeJobRun(ctx context.Context, store cache.Store, defaultBookDelay int`
`375`	`375`	`}`
`376`	`376`	`return nil, nil`
`377`	`377`	`}`
`378`		`- if h.ID == hatchery.ID {`
`379`		`- return nil, nil`
`380`		`- }`
`381`	`378`	`return &h, sdk.WrapError(sdk.ErrJobAlreadyBooked, "BookNodeJobRun> job %d already booked by %s (%d)", id, h.Name, h.ID)`
`382`	`379`	`}`
`383`	`380`