diff --git a/manager/manager.go b/manager/manager.go index 06e36ee8e0..cf6a8a1a10 100644 --- a/manager/manager.go +++ b/manager/manager.go @@ -932,7 +932,7 @@ func (m *manager) createContainerLocked(containerName string, watchSource watche perfCgroupPath := path.Join(fs2.UnifiedMountpoint, containerName) cont.perfCollector, err = m.perfManager.GetCollector(perfCgroupPath) if err != nil { - klog.V(4).Infof("perf_event metrics will not be available for container %s: %s", containerName, err) + klog.Errorf("Perf event metrics will not be available for container %q: %v", containerName, err) } } else { devicesCgroupPath, err := handler.GetCgroupPath("devices") @@ -950,7 +950,7 @@ func (m *manager) createContainerLocked(containerName string, watchSource watche } else { cont.perfCollector, err = m.perfManager.GetCollector(perfCgroupPath) if err != nil { - klog.V(4).Infof("perf_event metrics will not be available for container %s: %s", containerName, err) + klog.Errorf("Perf event metrics will not be available for container %q: %v", containerName, err) } } } diff --git a/perf/uncore_libpfm.go b/perf/uncore_libpfm.go index 2e4cf20d53..61346047d5 100644 --- a/perf/uncore_libpfm.go +++ b/perf/uncore_libpfm.go @@ -151,8 +151,7 @@ func NewUncoreCollector(cgroupPath string, events PerfEvents, cpuToSocket map[in err := collector.setup(events, systemDevicesPath) if err != nil { - formatedError := fmt.Errorf("unable to setup uncore perf event collector: %v", err) - klog.V(5).Infof("Perf uncore metrics will not be available: %s", formatedError) + klog.Errorf("Perf uncore metrics will not be available: unable to setup uncore perf event collector: %v", err) return &stats.NoopCollector{} }