refactor: replace Debugf with Debug for logging messages

Signed-off-by: fahed dorgaa <[email protected]>

Author: fahedouch

pkg/network/gvisortapvsock/gvisortapvsock.go

@@ -243,7 +243,7 @@ func (d *parentDriver) acceptConnections() {
 			if err != nil {
 				// Check if the error is due to the listener being closed, which is expected during cleanup
 				if errors.Is(err, io.EOF) || strings.Contains(err.Error(), "use of closed network connection") {
-					logrus.Debugf("listener closed, stopping accept loop")
+					logrus.Debug("listener closed, stopping accept loop")
 					return
 				}
 				logrus.Errorf("accepting connection: %v", err)
@@ -271,9 +271,9 @@ func (d *parentDriver) handleConnection(conn net.Conn) {
 
 	// Use the AcceptStdio function to stream packets between the Unix socket and the virtual network
 	// This will handle the connection and forward packets in both directions
-	logrus.Debugf("forwarding packets between Unix socket and virtual network using VfkitProtocol")
+	logrus.Debug("forwarding packets between Unix socket and virtual network using VfkitProtocol")
 	if err := vn.AcceptStdio(d.ctx, conn); err != nil {
-		logrus.Debugf(err.Error())
+		logrus.Debug(err)
 		if errors.Is(err, io.EOF) {
 			// This is expected when the child process exits
 			logrus.Debugf("child process exited, connection closed: %v", err)
@@ -286,7 +286,7 @@ func (d *parentDriver) handleConnection(conn net.Conn) {
 // createCleanupFunc creates a cleanup function for the virtual network
 func (d *parentDriver) createCleanupFunc(vn *virtualnetwork.VirtualNetwork) func() error {
 	return func() error {
-		logrus.Debugf("closing gvisor-tap-vsock virtual network")
+		logrus.Debug("closing gvisor-tap-vsock virtual network")
 		// The VirtualNetwork struct doesn't have an explicit Close method,
 		// but we'll keep a reference to it to prevent garbage collection
 		_ = vn
@@ -301,7 +301,7 @@ func (d *parentDriver) createCleanupFunc(vn *virtualnetwork.VirtualNetwork) func
 			}
 		}
 
-		logrus.Debugf("closed gvisor-tap-vsock virtual network")
+		logrus.Debug("closed gvisor-tap-vsock virtual network")
 		return nil
 	}
 }

pkg/network/slirp4netns/slirp4netns.go

@@ -226,7 +226,7 @@ func (d *parentDriver) ConfigureNetwork(childPID int, stateDir, detachedNetNSPat
 	}
 	cmd.ExtraFiles = append(cmd.ExtraFiles, readyW)
 	cleanups = append(cleanups, func() error {
-		logrus.Debugf("killing slirp4netns")
+		logrus.Debug("killing slirp4netns")
 		if cmd.Process != nil {
 			_ = cmd.Process.Kill()
 		}

pkg/network/vpnkit/vpnkit.go

@@ -97,7 +97,7 @@ func (d *parentDriver) ConfigureNetwork(childPID int, stateDir, detachedNetNSPat
 		Pdeathsig: syscall.SIGKILL,
 	}
 	cleanups = append(cleanups, func() error {
-		logrus.Debugf("killing vpnkit")
+		logrus.Debug("killing vpnkit")
 		vpnkitCancel()
 		wErr := vpnkitCmd.Wait()
 		logrus.Debugf("killed vpnkit: %v", wErr)
@@ -120,7 +120,7 @@ func (d *parentDriver) ConfigureNetwork(childPID int, stateDir, detachedNetNSPat
 	if err != nil {
 		return nil, common.Seq(cleanups), fmt.Errorf("connecting to %s with uuid %s: %w", vpnkitSocket, vifUUID, err)
 	}
-	logrus.Debugf("connected to VPNKit vmnet")
+	logrus.Debug("connected to VPNKit vmnet")
 	// TODO: support configuration
 	netmsg := messages.ParentInitNetworkDriverCompleted{
 		Dev:     d.ifname,

pkg/parent/parent.go

@@ -367,10 +367,10 @@ func getSubIDRanges(u *user.User, subidSource SubidSource) ([]idtools.SubIDRange
 	}
 	switch subidSource {
 	case SubidSourceStatic:
-		logrus.Debugf("subid-source: using the static source")
+		logrus.Debug("subid-source: using the static source")
 		return idtools.GetSubIDRanges(uid, u.Username)
 	case SubidSourceDynamic:
-		logrus.Debugf("subid-source: using the dynamic source")
+		logrus.Debug("subid-source: using the dynamic source")
 		return dynidtools.GetSubIDRanges(uid, u.Username)
 	case "", SubidSourceAuto:
 		subuidRanges, subgidRanges, err := getSubIDRanges(u, SubidSourceDynamic)

pkg/port/gvisortapvsock/gvisortapvsock.go

@@ -193,15 +193,15 @@ func (d *driver) RunParentDriver(initComplete chan struct{}, quit <-chan struct{
 	// Get the virtual network from the child context
 	if cctx != nil && cctx.Network != nil {
 		d.servicesMux = cctx.Network.Mux()
-		logrus.Debugf("Using services mux from child context")
+		logrus.Debug("Using services mux from child context")
 	}
 
 	if d.servicesMux == nil {
 		return fmt.Errorf("Virtual network services mux not available in child context")
 	}
 
 	d.mu.Unlock()
-	logrus.Debugf("Created virtual network")
+	logrus.Debug("Created virtual network")
 
 	// Signal that initialization is complete
 	close(initComplete)