Fix error formatting

Author: 0xThiebaut

metric/memory/memory_aix.go

@@ -57,7 +57,7 @@ func get(_ resolve.Resolver) (Memory, error) {
 	meminfo := C.perfstat_memory_total_t{}
 	_, err := C.perfstat_memory_total(nil, &meminfo, C.sizeof_perfstat_memory_total_t, 1)
 	if err != nil {
-		return memData, fmt.Errorf("perfstat_memory_total: %s", err)
+		return memData, fmt.Errorf("perfstat_memory_total: %w", err)
 	}
 
 	totalMem := uint64(meminfo.real_total) * system.pagesize

metric/system/cgroup/util_test.go

@@ -123,7 +123,7 @@ func TestSupportedSubsystems(t *testing.T) {
 func TestSupportedSubsystemsErrCgroupsMissing(t *testing.T) {
 	_, err := SupportedSubsystems(resolve.NewTestResolver("testdata/doesnotexist"))
 	if !errors.Is(err, ErrCgroupsMissing) {
-		t.Fatalf("expected ErrCgroupsMissing, but got %v", err)
+		t.Fatalf("expected ErrCgroupsMissing, but got %s", err)
 	}
 }
 

metric/system/diskio/diskstat_windows_helper.go

@@ -72,7 +72,7 @@ func ioCounters(names ...string) (map[string]disk.IOCountersStat, error) {
 		var counter diskPerformance
 		err = ioCounter(drive.UNCPath, &counter)
 		if err != nil {
-			logp.Err("Could not return any performance counter values for %s .Error: %v", drive.UNCPath, err)
+			logp.Errorf("Could not return any performance counter values for %s .Error: %s", drive.UNCPath, err)
 			continue
 		}
 		ret[drive.Name] = disk.IOCountersStat{

metric/system/filesystem/filesystem_test.go

@@ -44,7 +44,7 @@ func TestFileSystemList(t *testing.T) {
 
 	for _, fs := range fss {
 		err := fs.GetUsage()
-		assert.NoError(t, err, "filesystem=%#v: %v", fs, err)
+		assert.NoError(t, err, "filesystem=%#v: %s", fs, err)
 
 	}
 }

metric/system/process/process.go

@@ -62,7 +62,7 @@ func ListStates(hostfs resolve.Resolver) ([]ProcState, error) {
 	}
 	failedPIDs := extractFailedPIDs(pidMap)
 	if err != nil && len(failedPIDs) > 0 {
-		init.logger.Debugf("error fetching process metrics: %v", err)
+		init.logger.Debugf("error fetching process metrics: %s", err)
 		return plist, NonFatalErr{Err: fmt.Errorf(errFetchingPIDs, len(failedPIDs))}
 	}
 	return plist, toNonFatal(err)
@@ -115,7 +115,7 @@ func (procStats *Stats) Get() ([]mapstr.M, []mapstr.M, error) {
 	if procStats.host != nil {
 		memStats, err := procStats.host.Memory()
 		if err != nil {
-			procStats.logger.Warnf("Getting memory details: %v", err)
+			procStats.logger.Warnf("Getting memory details: %s", err)
 		} else {
 			totalPhyMem = memStats.Total
 		}
@@ -141,7 +141,7 @@ func (procStats *Stats) Get() ([]mapstr.M, []mapstr.M, error) {
 		rootEvents = append(rootEvents, rootMap)
 	}
 	if wrappedErr != nil && len(failedPIDs) > 0 {
-		procStats.logger.Debugf("error fetching process metrics: %v", wrappedErr)
+		procStats.logger.Debugf("error fetching process metrics: %s", wrappedErr)
 		return procs, rootEvents, NonFatalErr{Err: fmt.Errorf(errFetchingPIDs, len(failedPIDs))}
 	}
 	return procs, rootEvents, toNonFatal(wrappedErr)

metric/system/process/process_common.go

@@ -162,7 +162,7 @@ func (procStats *Stats) Init() error {
 	procStats.host, err = sysinfo.Host()
 	if err != nil {
 		procStats.host = nil
-		procStats.logger.Warnf("Getting host details: %v", err)
+		procStats.logger.Warnf("Getting host details: %s", err)
 	}
 
 	// footcannon prevention
@@ -201,7 +201,7 @@ func (procStats *Stats) Init() error {
 	if procStats.EnableCgroups {
 		cgReader, err := cgroup.NewReaderOptions(procStats.CgroupOpts)
 		if errors.Is(err, cgroup.ErrCgroupsMissing) {
-			logp.Warn("cgroup data collection will be disabled: %v", err)
+			logp.Warnf("cgroup data collection will be disabled: %s", err)
 			procStats.EnableCgroups = false
 		} else if err != nil {
 			return fmt.Errorf("error initializing cgroup reader: %w", err)

metric/system/process/process_linux_darwin_test.go

@@ -40,7 +40,7 @@ func TestGetInfoForPid_numThreads(t *testing.T) {
 	if !got.NumThreads.Exists() {
 		bs, err := json.Marshal(got)
 		if err != nil {
-			t.Logf("could not marshal ProcState: %v", err)
+			t.Logf("could not marshal ProcState: %s", err)
 		}
 		t.Fatalf("num_thread was not collected. Collected info: %s", bs)
 	}

metric/system/process/process_test.go

@@ -115,7 +115,7 @@ func TestGetState(t *testing.T) {
 
 	assert.Eventuallyf(t, test,
 		time.Second*5, 50*time.Millisecond,
-		"got process state %q. Last error: %v", got, err)
+		"got process state %q. Last error: %s", got, err)
 }
 
 func TestGetOneRoot(t *testing.T) {

metric/system/process/process_windows_test.go

@@ -54,7 +54,7 @@ func TestGetInfoForPid_numThreads(t *testing.T) {
 	if !state.NumThreads.Exists() {
 		bs, err := json.Marshal(state)
 		if err != nil {
-			t.Logf("could not marshal ProcState: %v", err)
+			t.Logf("could not marshal ProcState: %s", err)
 		}
 		t.Fatalf("num_thread was not collected. Collected info: %s", bs)
 	}

report/metrics_file_descriptors.go

@@ -37,7 +37,7 @@ func FDUsageReporter(logger *logp.Logger, processStats *process.Stats) func(_ mo
 
 		open, hardLimit, softLimit, err := getFDUsage(processStats)
 		if err != nil {
-			logger.Error("Error while retrieving FD information: %v", err)
+			logger.Errorf("Error while retrieving FD information: %s", err)
 			return
 		}