Merge pull request #389 from pace/routine-cancellation

pkg/routine: Use WithoutCancel instead of Transfer

pkg/routine/routine.go

@@ -15,7 +15,6 @@ import (
 	"github.com/getsentry/sentry-go"
 	"github.com/pace/bricks/maintenance/errors"
 	"github.com/pace/bricks/maintenance/log"
-	pkgcontext "github.com/pace/bricks/pkg/context"
 )
 
 type options struct {
@@ -82,13 +81,12 @@ func RunNamed(parentCtx context.Context, name string, routine func(context.Conte
 	return Run(parentCtx, routine)
 }
 
-// Run runs the given function in a new background context. The new context
-// inherits the logger and oauth2 authentication of the parent context. Panics
+// Run runs the given function in a new background context. Panics
 // thrown in the function are logged and sent to sentry. The routines context is
 // canceled if the program receives a shutdown signal (SIGINT, SIGTERM), if the
 // returned CancelFunc is called, or if the routine returned.
-func Run(parentCtx context.Context, routine func(context.Context)) (cancel context.CancelFunc) {
-	ctx := pkgcontext.Transfer(parentCtx)
+func Run(ctx context.Context, routine func(context.Context)) (cancel context.CancelFunc) {
+	ctx = context.WithoutCancel(ctx)
 
 	// add routine number to context and logger
 	num := atomic.AddInt64(&ctr, 1)