github/containerd
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 2 Wiki Activity

ctr: metrics: drop unused retval from printWindowsStats()

Browse Source 
The error return value is always nil, so we don't need it at all.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
This commit is contained in:
Enrico Weigelt, metux IT consult 2023-08-16 12:33:04 +02:00
parent 465c04c289
commit 1571a5aa07
1 changed files with 2 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
cmd/ctr/commands/tasks/metrics.go
View File

@ -94,9 +94,7 @@ var metricsCommand = cli.Command{
case *v2.Metrics: case *v2.Metrics:
printCgroup2MetricsTable(w, v) printCgroup2MetricsTable(w, v)
case *wstats.Statistics: case *wstats.Statistics:
if err := printWindowsStats(w, v); err != nil { printWindowsStats(w, v)
return fmt.Errorf("cannot convert metrics data from windows.Statistics: %w", err)
}
} }
return w.Flush() return w.Flush()
case formatJSON: case formatJSON:
@ -151,7 +149,7 @@ func printCgroup2MetricsTable(w *tabwriter.Writer, data *v2.Metrics) {
} }
} }
func printWindowsStats(w *tabwriter.Writer, windowsStats *wstats.Statistics) error { func printWindowsStats(w *tabwriter.Writer, windowsStats *wstats.Statistics) {
if windowsStats.GetLinux() != nil { if windowsStats.GetLinux() != nil {
stats := windowsStats.GetLinux() stats := windowsStats.GetLinux()
printCgroupMetricsTable(w, stats) printCgroupMetricsTable(w, stats)
@ -162,7 +160,6 @@ func printWindowsStats(w *tabwriter.Writer, windowsStats *wstats.Statistics) err
if windowsStats.VM != nil { if windowsStats.VM != nil {
printWindowsVMStatistics(w, windowsStats.VM) printWindowsVMStatistics(w, windowsStats.VM)
} }
return nil
} }
func printWindowsContainerStatistics(w *tabwriter.Writer, stats *wstats.WindowsContainerStatistics) { func printWindowsContainerStatistics(w *tabwriter.Writer, stats *wstats.WindowsContainerStatistics) {