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()
 

Reply via email to