fix: remove unnecessary loop var capture (Go 1.22+)

Go 1.22+ changed loop variable semantics — each iteration gets its
own copy. No need to capture deviceIndex/i before the goroutine.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: 2 people

packages/orchestrator/pkg/sandbox/nbd/path_direct.go

@@ -119,18 +119,14 @@ func (d *DirectPathMount) Open(ctx context.Context) (retDeviceIndex uint32, err
 			server.Close()
 
 			dispatch := NewDispatch(serverc, d.Backend)
-			// Capture loop variables for the goroutine closure to avoid a data
-			// race: deviceIndex is reassigned on each retry iteration of the
-			// outer for-loop while the goroutine may still read it.
-			devIdx := deviceIndex
 			// Start reading commands on the socket and dispatching them to our provider
 			d.handlersWg.Go(func() {
 				handleErr := dispatch.Handle(ctx)
 				// The error is expected to happen if the nbd (socket connection) is closed
 				logger.L().Info(ctx, "closing handler for NBD commands",
 					zap.Error(handleErr),
-					zap.Uint32("device_index", devIdx),
-					zap.Int("socket_index", sockIdx),
+					zap.Uint32("device_index", deviceIndex),
+					zap.Int("socket_index", i),
 				)
 			})