Consistently use WithError when logging errors instead of %s

Signed-off-by: apostasie <[email protected]>

Author: apostasie

cmd/nerdctl/container/container_run.go

@@ -386,7 +386,7 @@ func runAction(cmd *cobra.Command, args []string) error {
 				return
 			}
 			if err := netManager.CleanupNetworking(ctx, c); err != nil {
-				log.L.Warnf("failed to clean up container networking: %s", err)
+				log.L.WithError(err).Warnf("failed to clean up container networking")
 			}
 			if err := container.RemoveContainer(ctx, c, createOpt.GOptions, true, true, client); err != nil {
 				log.L.WithError(err).Warnf("failed to remove container %s", id)

pkg/infoutil/infoutil.go

@@ -160,13 +160,13 @@ func ServerSemVer(ctx context.Context, client *containerd.Client) (*semver.Versi
 func buildctlVersion() dockercompat.ComponentVersion {
 	buildctlBinary, err := buildkitutil.BuildctlBinary()
 	if err != nil {
-		log.L.Warnf("unable to determine buildctl version: %s", err.Error())
+		log.L.WithError(err).Warnf("unable to determine buildctl version")
 		return dockercompat.ComponentVersion{Name: "buildctl"}
 	}
 
 	stdout, err := exec.Command(buildctlBinary, "--version").Output()
 	if err != nil {
-		log.L.Warnf("unable to determine buildctl version: %s", err.Error())
+		log.L.WithError(err).Warnf("unable to determine buildctl version")
 		return dockercompat.ComponentVersion{Name: "buildctl"}
 	}
 
@@ -205,7 +205,7 @@ func parseBuildctlVersion(buildctlVersionStdout []byte) (*dockercompat.Component
 func runcVersion() dockercompat.ComponentVersion {
 	stdout, err := exec.Command("runc", "--version").Output()
 	if err != nil {
-		log.L.Warnf("unable to determine runc version: %s", err.Error())
+		log.L.WithError(err).Warnf("unable to determine runc version")
 		return dockercompat.ComponentVersion{Name: "runc"}
 	}
 	v, err := parseRuncVersion(stdout)

pkg/logging/jsonfile/jsonfile.go

@@ -146,7 +146,7 @@ func Decode(stdout, stderr io.Writer, r io.Reader, timestamps bool, since string
 		// Write out the entry directly
 		err := writeEntry(&e, stdout, stderr, now, timestamps, since, until)
 		if err != nil {
-			log.L.Errorf("error while writing log entry to output stream: %s", err)
+			log.L.WithError(err).Errorf("error while writing log entry to output stream")
 		}
 	}
 

pkg/logging/logging.go

@@ -264,7 +264,7 @@ func startTail(ctx context.Context, logName string, w *fsnotify.Watcher) (bool,
 				log.L.Debugf("Received unexpected fsnotify event: %v, retrying", e)
 			}
 		case err := <-w.Errors:
-			log.L.Debugf("Received fsnotify watch error, retrying unless no more retries left, retries: %d, error: %s", errRetry, err)
+			log.L.WithError(err).Debugf("Received fsnotify watch error, retrying unless no more retries left, retries: %d", errRetry)
 			if errRetry == 0 {
 				return false, err
 			}