This is an automated email from the ASF dual-hosted git repository.
astefanutti pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel-k.git
The following commit(s) were added to refs/heads/main by this push:
new 4516138c2 chore: Propagate reconcilation context to build routine
4516138c2 is described below
commit 4516138c288ec08f06fc28475efa3cd7f83655f1
Author: Antonin Stefanutti <[email protected]>
AuthorDate: Tue Sep 27 11:12:45 2022 +0200
chore: Propagate reconcilation context to build routine
---
pkg/controller/build/monitor_routine.go | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/pkg/controller/build/monitor_routine.go
b/pkg/controller/build/monitor_routine.go
index 055d960bd..42f5a7902 100644
--- a/pkg/controller/build/monitor_routine.go
+++ b/pkg/controller/build/monitor_routine.go
@@ -77,8 +77,7 @@ func (action *monitorRoutineAction) Handle(ctx
context.Context, build *v1.Build)
// Start the build asynchronously to avoid blocking the
reconciliation loop
routines.Store(build.Name, true)
- // nolint: contextcheck
- go action.runBuild(build)
+ go action.runBuild(ctx, build)
case v1.BuildPhaseRunning:
if _, ok := routines.Load(build.Name); !ok {
@@ -93,10 +92,9 @@ func (action *monitorRoutineAction) Handle(ctx
context.Context, build *v1.Build)
return nil, nil
}
-func (action *monitorRoutineAction) runBuild(build *v1.Build) {
+func (action *monitorRoutineAction) runBuild(ctx context.Context, build
*v1.Build) {
defer routines.Delete(build.Name)
- ctx := context.Background()
ctxWithTimeout, cancel := context.WithDeadline(ctx,
build.Status.StartedAt.Add(build.Spec.Timeout.Duration))
defer cancel()