remove unneeded nolint-comments (nolintlint), disable deprecated linters

thaJeztah · thaJeztah · commit f9c80be1bbc0 · 2022-10-12T14:41:01.000+02:00
Remove nolint-comments that weren't hit by linters, and remove the "structcheck" and "varcheck" linters, as they have been deprecated: WARN [runner] The linter 'structcheck' is deprecated (since v1.49.0) due to: The owner seems to have abandoned the linter. Replaced by unused. WARN [runner] The linter 'varcheck' is deprecated (since v1.49.0) due to: The owner seems to have abandoned the linter. Replaced by unused. WARN [linters context] structcheck is disabled because of generics. You can track the evolution of the generics support by following the golangci/golangci-lint#2649. Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
diff --git a/.golangci.yml b/.golangci.yml
@@ -6,13 +6,12 @@ linters:
     - gosec
     - ineffassign
     - misspell
+    - nolintlint
     - revive
     - staticcheck
-    - structcheck
     - tenv # Detects using os.Setenv instead of t.Setenv since Go 1.17
     - unconvert
     - unused
-    - varcheck
     - vet
   disable:
     - errcheck
diff --git a/archive/tar_unix.go b/archive/tar_unix.go
@@ -58,7 +58,7 @@ func setHeaderForSpecialDevice(hdr *tar.Header, name string, fi os.FileInfo) err
 		return errors.New("unsupported stat type")
 	}
 
-	rdev := uint64(s.Rdev) //nolint:unconvert // rdev is int32 on darwin/bsd, int64 on linux/solaris
+	rdev := uint64(s.Rdev) //nolint:nolintlint,unconvert // rdev is int32 on darwin/bsd, int64 on linux/solaris
 
 	// Currently go does not fill in the major/minors
 	if s.Mode&syscall.S_IFBLK != 0 ||
diff --git a/integration/main_test.go b/integration/main_test.go
@@ -210,7 +210,7 @@ func PodSandboxConfigWithCleanup(t *testing.T, name, ns string, opts ...PodSandb
 }
 
 // Set Windows HostProcess on the pod.
-func WithWindowsHostProcessPod(p *runtime.PodSandboxConfig) { //nolint:unused
+func WithWindowsHostProcessPod(p *runtime.PodSandboxConfig) {
 	if p.Windows == nil {
 		p.Windows = &runtime.WindowsPodSandboxConfig{}
 	}
@@ -237,7 +237,7 @@ func WithTestAnnotations() ContainerOpts {
 }
 
 // Add container resource limits.
-func WithResources(r *runtime.LinuxContainerResources) ContainerOpts { //nolint:unused
+func WithResources(r *runtime.LinuxContainerResources) ContainerOpts {
 	return func(c *runtime.ContainerConfig) {
 		if c.Linux == nil {
 			c.Linux = &runtime.LinuxContainerConfig{}
@@ -247,7 +247,7 @@ func WithResources(r *runtime.LinuxContainerResources) ContainerOpts { //nolint:
 }
 
 // Adds Windows container resource limits.
-func WithWindowsResources(r *runtime.WindowsContainerResources) ContainerOpts { //nolint:unused
+func WithWindowsResources(r *runtime.WindowsContainerResources) ContainerOpts {
 	return func(c *runtime.ContainerConfig) {
 		if c.Windows == nil {
 			c.Windows = &runtime.WindowsContainerConfig{}
@@ -265,7 +265,7 @@ func WithVolumeMount(hostPath, containerPath string) ContainerOpts {
 	}
 }
 
-func WithWindowsUsername(username string) ContainerOpts { //nolint:unused
+func WithWindowsUsername(username string) ContainerOpts {
 	return func(c *runtime.ContainerConfig) {
 		if c.Windows == nil {
 			c.Windows = &runtime.WindowsContainerConfig{}
@@ -277,7 +277,7 @@ func WithWindowsUsername(username string) ContainerOpts { //nolint:unused
 	}
 }
 
-func WithWindowsHostProcessContainer() ContainerOpts { //nolint:unused
+func WithWindowsHostProcessContainer() ContainerOpts {
 	return func(c *runtime.ContainerConfig) {
 		if c.Windows == nil {
 			c.Windows = &runtime.WindowsContainerConfig{}
@@ -322,7 +322,7 @@ func WithLogPath(path string) ContainerOpts {
 }
 
 // WithSupplementalGroups adds supplemental groups.
-func WithSupplementalGroups(gids []int64) ContainerOpts { //nolint:unused
+func WithSupplementalGroups(gids []int64) ContainerOpts {
 	return func(c *runtime.ContainerConfig) {
 		if c.Linux == nil {
 			c.Linux = &runtime.LinuxContainerConfig{}
@@ -335,7 +335,7 @@ func WithSupplementalGroups(gids []int64) ContainerOpts { //nolint:unused
 }
 
 // WithDevice adds a device mount.
-func WithDevice(containerPath, hostPath, permissions string) ContainerOpts { //nolint:unused
+func WithDevice(containerPath, hostPath, permissions string) ContainerOpts {
 	return func(c *runtime.ContainerConfig) {
 		c.Devices = append(c.Devices, &runtime.Device{
 			ContainerPath: containerPath, HostPath: hostPath, Permissions: permissions,
@@ -558,7 +558,7 @@ func CRIConfig() (*criconfig.Config, error) {
 }
 
 // SandboxInfo gets sandbox info.
-func SandboxInfo(id string) (*runtime.PodSandboxStatus, *server.SandboxInfo, error) { //nolint:unused
+func SandboxInfo(id string) (*runtime.PodSandboxStatus, *server.SandboxInfo, error) {
 	client, err := RawRuntimeClient()
 	if err != nil {
 		return nil, nil, fmt.Errorf("failed to get raw runtime client: %w", err)
diff --git a/oci/spec_opts.go b/oci/spec_opts.go
@@ -84,7 +84,7 @@ func setResources(s *Spec) {
 	}
 }
 
-//nolint:unused // not used on all platforms
+//nolint:nolintlint,unused // not used on all platforms
 func setResourcesWindows(s *Spec) {
 	if s.Windows != nil {
 		if s.Windows.Resources == nil {
@@ -93,7 +93,7 @@ func setResourcesWindows(s *Spec) {
 	}
 }
 
-//nolint:unused // not used on all platforms
+//nolint:nolintlint,unused // not used on all platforms
 func setCPU(s *Spec) {
 	setResources(s)
 	if s.Linux != nil {
@@ -103,7 +103,7 @@ func setCPU(s *Spec) {
 	}
 }
 
-//nolint:deadcode,unused // not used on all platforms
+//nolint:deadcode,nolintlint,unused // not used on all platforms
 func setCPUWindows(s *Spec) {
 	setResourcesWindows(s)
 	if s.Windows != nil {
diff --git a/oci/spec_opts_nonlinux.go b/oci/spec_opts_nonlinux.go
@@ -28,31 +28,23 @@ import (
 
 // WithAllCurrentCapabilities propagates the effective capabilities of the caller process to the container process.
 // The capability set may differ from WithAllKnownCapabilities when running in a container.
-//
-//nolint:unused
 var WithAllCurrentCapabilities = func(ctx context.Context, client Client, c *containers.Container, s *Spec) error {
 	return WithCapabilities(nil)(ctx, client, c, s)
 }
 
 // WithAllKnownCapabilities sets all the known linux capabilities for the container process
-//
-//nolint:unused
 var WithAllKnownCapabilities = func(ctx context.Context, client Client, c *containers.Container, s *Spec) error {
 	return WithCapabilities(nil)(ctx, client, c, s)
 }
 
 // WithBlockIO sets the container's blkio parameters
-//
-//nolint:unused
 func WithBlockIO(blockio interface{}) SpecOpts {
 	return func(ctx context.Context, _ Client, c *containers.Container, s *Spec) error {
 		return errors.New("blkio not supported")
 	}
 }
 
 // WithCPUShares sets the container's cpu shares
-//
-//nolint:unused
 func WithCPUShares(shares uint64) SpecOpts {
 	return func(ctx context.Context, _ Client, c *containers.Container, s *Spec) error {
 		return nil
diff --git a/oci/utils_unix.go b/oci/utils_unix.go
@@ -127,7 +127,7 @@ func getDevices(path, containerPath string) ([]specs.LinuxDevice, error) {
 
 // TODO consider adding these consts to the OCI runtime-spec.
 const (
-	wildcardDevice = "a" //nolint:deadcode,unused,varcheck // currently unused, but should be included when upstreaming to OCI runtime-spec.
+	wildcardDevice = "a" //nolint:deadcode,nolintlint,unused,varcheck // currently unused, but should be included when upstreaming to OCI runtime-spec.
 	blockDevice    = "b"
 	charDevice     = "c" // or "u"
 	fifoDevice     = "p"
@@ -148,7 +148,7 @@ func DeviceFromPath(path string) (*specs.LinuxDevice, error) {
 	}
 
 	var (
-		devNumber = uint64(stat.Rdev) //nolint:unconvert // the type is 32bit on mips.
+		devNumber = uint64(stat.Rdev) //nolint:nolintlint,unconvert // the type is 32bit on mips.
 		major     = unix.Major(devNumber)
 		minor     = unix.Minor(devNumber)
 	)
diff --git a/pkg/cri/sbserver/container_stats.go b/pkg/cri/sbserver/container_stats.go
@@ -41,7 +41,7 @@ func (c *criService) ContainerStats(ctx context.Context, in *runtime.ContainerSt
 	}
 
 	cs, err := c.containerMetrics(cntr.Metadata, resp.Metrics[0])
-	if err != nil { //nolint:staticcheck // Ignore SA4023 as some platforms always return nil (stats unimplemented)
+	if err != nil {
 		return nil, fmt.Errorf("failed to decode container metrics: %w", err)
 	}
 	return &runtime.ContainerStatsResponse{Stats: cs}, nil
diff --git a/pkg/cri/sbserver/container_stats_list.go b/pkg/cri/sbserver/container_stats_list.go
@@ -58,7 +58,7 @@ func (c *criService) toCRIContainerStats(
 	containerStats := new(runtime.ListContainerStatsResponse)
 	for _, cntr := range containers {
 		cs, err := c.containerMetrics(cntr.Metadata, statsMap[cntr.ID])
-		if err != nil { //nolint:staticcheck // Ignore SA4023 as some platforms always return nil (metrics unimplemented)
+		if err != nil {
 			return nil, fmt.Errorf("failed to decode container metrics for %q: %w", cntr.ID, err)
 		}
 		containerStats.Stats = append(containerStats.Stats, cs)
diff --git a/pkg/cri/sbserver/sandbox_stats.go b/pkg/cri/sbserver/sandbox_stats.go
@@ -34,12 +34,12 @@ func (c *criService) PodSandboxStats(
 	}
 
 	metrics, err := metricsForSandbox(sandbox)
-	if err != nil { //nolint:staticcheck // Ignore SA4023 as some platforms always return nil (unimplemented metrics)
+	if err != nil {
 		return nil, fmt.Errorf("failed getting metrics for sandbox %s: %w", r.GetPodSandboxId(), err)
 	}
 
 	podSandboxStats, err := c.podSandboxStats(ctx, sandbox, metrics)
-	if err != nil { //nolint:staticcheck // Ignore SA4023 as some platforms always return nil (unimplemented metrics)
+	if err != nil {
 		return nil, fmt.Errorf("failed to decode pod sandbox metrics %s: %w", r.GetPodSandboxId(), err)
 	}
 
diff --git a/pkg/cri/sbserver/sandbox_stats_list.go b/pkg/cri/sbserver/sandbox_stats_list.go
@@ -34,12 +34,12 @@ func (c *criService) ListPodSandboxStats(
 	podSandboxStats := new(runtime.ListPodSandboxStatsResponse)
 	for _, sandbox := range sandboxes {
 		metrics, err := metricsForSandbox(sandbox)
-		if err != nil { //nolint:staticcheck // Ignore SA4023 as some platforms always return nil (unimplemented metrics)
+		if err != nil {
 			return nil, fmt.Errorf("failed to obtain metrics for sandbox %q: %w", sandbox.ID, err)
 		}
 
 		sandboxStats, err := c.podSandboxStats(ctx, sandbox, metrics)
-		if err != nil { //nolint:staticcheck // Ignore SA4023 as some platforms always return nil (unimplemented metrics)
+		if err != nil {
 			return nil, fmt.Errorf("failed to decode sandbox container metrics for sandbox %q: %w", sandbox.ID, err)
 		}
 		podSandboxStats.Stats = append(podSandboxStats.Stats, sandboxStats)
diff --git a/pkg/cri/sbserver/service.go b/pkg/cri/sbserver/service.go
@@ -117,7 +117,7 @@ type criService struct {
 	baseOCISpecs map[string]*oci.Spec
 	// allCaps is the list of the capabilities.
 	// When nil, parsed from CapEff of /proc/self/status.
-	allCaps []string //nolint:unused // Ignore on non-Linux
+	allCaps []string //nolint:nolintlint,unused // Ignore on non-Linux
 	// unpackDuplicationSuppressor is used to make sure that there is only
 	// one in-flight fetch request or unpack handler for a given descriptor's
 	// or chain ID.
diff --git a/pkg/cri/server/container_stats.go b/pkg/cri/server/container_stats.go
@@ -41,7 +41,7 @@ func (c *criService) ContainerStats(ctx context.Context, in *runtime.ContainerSt
 	}
 
 	cs, err := c.containerMetrics(cntr.Metadata, resp.Metrics[0])
-	if err != nil { //nolint:staticcheck // Ignore SA4023 as some platforms always return nil (stats unimplemented)
+	if err != nil {
 		return nil, fmt.Errorf("failed to decode container metrics: %w", err)
 	}
 	return &runtime.ContainerStatsResponse{Stats: cs}, nil
diff --git a/pkg/cri/server/container_stats_list.go b/pkg/cri/server/container_stats_list.go
@@ -61,7 +61,7 @@ func (c *criService) toCRIContainerStats(
 	containerStats := new(runtime.ListContainerStatsResponse)
 	for _, cntr := range containers {
 		cs, err := c.containerMetrics(cntr.Metadata, statsMap[cntr.ID])
-		if err != nil { //nolint:staticcheck // Ignore SA4023 as some platforms always return nil (metrics unimplemented)
+		if err != nil {
 			return nil, fmt.Errorf("failed to decode container metrics for %q: %w", cntr.ID, err)
 		}
 
diff --git a/pkg/cri/server/sandbox_stats.go b/pkg/cri/server/sandbox_stats.go
@@ -34,12 +34,12 @@ func (c *criService) PodSandboxStats(
 	}
 
 	metrics, err := metricsForSandbox(sandbox)
-	if err != nil { //nolint:staticcheck // Ignore SA4023 as some platforms always return nil (unimplemented metrics)
+	if err != nil {
 		return nil, fmt.Errorf("failed getting metrics for sandbox %s: %w", r.GetPodSandboxId(), err)
 	}
 
 	podSandboxStats, err := c.podSandboxStats(ctx, sandbox, metrics)
-	if err != nil { //nolint:staticcheck // Ignore SA4023 as some platforms always return nil (unimplemented metrics)
+	if err != nil {
 		return nil, fmt.Errorf("failed to decode pod sandbox metrics %s: %w", r.GetPodSandboxId(), err)
 	}
 
diff --git a/pkg/cri/server/sandbox_stats_list.go b/pkg/cri/server/sandbox_stats_list.go
@@ -34,12 +34,12 @@ func (c *criService) ListPodSandboxStats(
 	podSandboxStats := new(runtime.ListPodSandboxStatsResponse)
 	for _, sandbox := range sandboxes {
 		metrics, err := metricsForSandbox(sandbox)
-		if err != nil { //nolint:staticcheck // Ignore SA4023 as some platforms always return nil (unimplemented metrics)
+		if err != nil {
 			return nil, fmt.Errorf("failed to obtain metrics for sandbox %q: %w", sandbox.ID, err)
 		}
 
 		sandboxStats, err := c.podSandboxStats(ctx, sandbox, metrics)
-		if err != nil { //nolint:staticcheck // Ignore SA4023 as some platforms always return nil (unimplemented metrics)
+		if err != nil {
 			return nil, fmt.Errorf("failed to decode sandbox container metrics for sandbox %q: %w", sandbox.ID, err)
 		}
 		podSandboxStats.Stats = append(podSandboxStats.Stats, sandboxStats)
diff --git a/pkg/cri/server/service.go b/pkg/cri/server/service.go
@@ -113,7 +113,7 @@ type criService struct {
 	baseOCISpecs map[string]*oci.Spec
 	// allCaps is the list of the capabilities.
 	// When nil, parsed from CapEff of /proc/self/status.
-	allCaps []string //nolint:unused // Ignore on non-Linux
+	allCaps []string //nolint:nolintlint,unused // Ignore on non-Linux
 	// unpackDuplicationSuppressor is used to make sure that there is only
 	// one in-flight fetch request or unpack handler for a given descriptor's
 	// or chain ID.
diff --git a/pkg/cri/store/container/metadata.go b/pkg/cri/store/container/metadata.go
@@ -28,10 +28,9 @@ import (
 // 2) Metadata is checkpointed as containerd container label.
 
 // metadataVersion is current version of container metadata.
-const metadataVersion = "v1" // nolint
+const metadataVersion = "v1"
 
 // versionedMetadata is the internal versioned container metadata.
-// nolint
 type versionedMetadata struct {
 	// Version indicates the version of the versioned container metadata.
 	Version string
diff --git a/pkg/cri/store/container/status.go b/pkg/cri/store/container/status.go
@@ -61,10 +61,9 @@ import (
 //                      DELETED
 
 // statusVersion is current version of container status.
-const statusVersion = "v1" // nolint
+const statusVersion = "v1"
 
 // versionedStatus is the internal used versioned container status.
-// nolint
 type versionedStatus struct {
 	// Version indicates the version of the versioned container status.
 	Version string
diff --git a/pkg/cri/store/sandbox/metadata.go b/pkg/cri/store/sandbox/metadata.go
@@ -29,10 +29,9 @@ import (
 // 2) Metadata is checkpointed as containerd container label.
 
 // metadataVersion is current version of sandbox metadata.
-const metadataVersion = "v1" // nolint
+const metadataVersion = "v1"
 
 // versionedMetadata is the internal versioned sandbox metadata.
-// nolint
 type versionedMetadata struct {
 	// Version indicates the version of the versioned sandbox metadata.
 	Version string
diff --git a/pkg/progress/escape.go b/pkg/progress/escape.go
@@ -19,6 +19,6 @@ package progress
 const (
 	escape = "\x1b"
 	reset  = escape + "[0m"
-	red    = escape + "[31m" //nolint:deadcode,unused,varcheck
+	red    = escape + "[31m" //nolint:deadcode,nolintlint,unused,varcheck
 	green  = escape + "[32m"
 )

Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,7 @@ func setHeaderForSpecialDevice(hdr *tar.Header, name string, fi os.FileInfo) err`
`58`	`58`	`return errors.New("unsupported stat type")`
`59`	`59`	`}`
`60`	`60`
`61`		`- rdev := uint64(s.Rdev) //nolint:unconvert // rdev is int32 on darwin/bsd, int64 on linux/solaris`
	`61`	`+ rdev := uint64(s.Rdev) //nolint:nolintlint,unconvert // rdev is int32 on darwin/bsd, int64 on linux/solaris`
`62`	`62`
`63`	`63`	`// Currently go does not fill in the major/minors`
`64`	`64`	`if s.Mode&syscall.S_IFBLK != 0 \|\|`
Original file line number	Diff line number	Diff line change
`@@ -210,7 +210,7 @@ func PodSandboxConfigWithCleanup(t *testing.T, name, ns string, opts ...PodSandb`
`210`	`210`	`}`
`211`	`211`
`212`	`212`	`// Set Windows HostProcess on the pod.`
`213`		`-func WithWindowsHostProcessPod(p *runtime.PodSandboxConfig) { //nolint:unused`
	`213`	`+func WithWindowsHostProcessPod(p *runtime.PodSandboxConfig) {`
`214`	`214`	`if p.Windows == nil {`
`215`	`215`	`p.Windows = &runtime.WindowsPodSandboxConfig{}`
`216`	`216`	`}`
`@@ -237,7 +237,7 @@ func WithTestAnnotations() ContainerOpts {`
`237`	`237`	`}`
`238`	`238`
`239`	`239`	`// Add container resource limits.`
`240`		`-func WithResources(r *runtime.LinuxContainerResources) ContainerOpts { //nolint:unused`
	`240`	`+func WithResources(r *runtime.LinuxContainerResources) ContainerOpts {`
`241`	`241`	`return func(c *runtime.ContainerConfig) {`
`242`	`242`	`if c.Linux == nil {`
`243`	`243`	`c.Linux = &runtime.LinuxContainerConfig{}`
`@@ -247,7 +247,7 @@ func WithResources(r *runtime.LinuxContainerResources) ContainerOpts { //nolint:`
`247`	`247`	`}`
`248`	`248`
`249`	`249`	`// Adds Windows container resource limits.`
`250`		`-func WithWindowsResources(r *runtime.WindowsContainerResources) ContainerOpts { //nolint:unused`
	`250`	`+func WithWindowsResources(r *runtime.WindowsContainerResources) ContainerOpts {`
`251`	`251`	`return func(c *runtime.ContainerConfig) {`
`252`	`252`	`if c.Windows == nil {`
`253`	`253`	`c.Windows = &runtime.WindowsContainerConfig{}`
`@@ -265,7 +265,7 @@ func WithVolumeMount(hostPath, containerPath string) ContainerOpts {`
`265`	`265`	`}`
`266`	`266`	`}`
`267`	`267`
`268`		`-func WithWindowsUsername(username string) ContainerOpts { //nolint:unused`
	`268`	`+func WithWindowsUsername(username string) ContainerOpts {`
`269`	`269`	`return func(c *runtime.ContainerConfig) {`
`270`	`270`	`if c.Windows == nil {`
`271`	`271`	`c.Windows = &runtime.WindowsContainerConfig{}`
`@@ -277,7 +277,7 @@ func WithWindowsUsername(username string) ContainerOpts { //nolint:unused`
`277`	`277`	`}`
`278`	`278`	`}`
`279`	`279`
`280`		`-func WithWindowsHostProcessContainer() ContainerOpts { //nolint:unused`
	`280`	`+func WithWindowsHostProcessContainer() ContainerOpts {`
`281`	`281`	`return func(c *runtime.ContainerConfig) {`
`282`	`282`	`if c.Windows == nil {`
`283`	`283`	`c.Windows = &runtime.WindowsContainerConfig{}`
`@@ -322,7 +322,7 @@ func WithLogPath(path string) ContainerOpts {`
`322`	`322`	`}`
`323`	`323`
`324`	`324`	`// WithSupplementalGroups adds supplemental groups.`
`325`		`-func WithSupplementalGroups(gids []int64) ContainerOpts { //nolint:unused`
	`325`	`+func WithSupplementalGroups(gids []int64) ContainerOpts {`
`326`	`326`	`return func(c *runtime.ContainerConfig) {`
`327`	`327`	`if c.Linux == nil {`
`328`	`328`	`c.Linux = &runtime.LinuxContainerConfig{}`
`@@ -335,7 +335,7 @@ func WithSupplementalGroups(gids []int64) ContainerOpts { //nolint:unused`
`335`	`335`	`}`
`336`	`336`
`337`	`337`	`// WithDevice adds a device mount.`
`338`		`-func WithDevice(containerPath, hostPath, permissions string) ContainerOpts { //nolint:unused`
	`338`	`+func WithDevice(containerPath, hostPath, permissions string) ContainerOpts {`
`339`	`339`	`return func(c *runtime.ContainerConfig) {`
`340`	`340`	`c.Devices = append(c.Devices, &runtime.Device{`
`341`	`341`	`ContainerPath: containerPath, HostPath: hostPath, Permissions: permissions,`
`@@ -558,7 +558,7 @@ func CRIConfig() (*criconfig.Config, error) {`
`558`	`558`	`}`
`559`	`559`
`560`	`560`	`// SandboxInfo gets sandbox info.`
`561`		`-func SandboxInfo(id string) (runtime.PodSandboxStatus, server.SandboxInfo, error) { //nolint:unused`
	`561`	`+func SandboxInfo(id string) (runtime.PodSandboxStatus, server.SandboxInfo, error) {`
`562`	`562`	`client, err := RawRuntimeClient()`
`563`	`563`	`if err != nil {`
`564`	`564`	`return nil, nil, fmt.Errorf("failed to get raw runtime client: %w", err)`
Original file line number	Diff line number	Diff line change
`@@ -84,7 +84,7 @@ func setResources(s *Spec) {`
`84`	`84`	`}`
`85`	`85`	`}`
`86`	`86`
`87`		`-//nolint:unused // not used on all platforms`
	`87`	`+//nolint:nolintlint,unused // not used on all platforms`
`88`	`88`	`func setResourcesWindows(s *Spec) {`
`89`	`89`	`if s.Windows != nil {`
`90`	`90`	`if s.Windows.Resources == nil {`
`@@ -93,7 +93,7 @@ func setResourcesWindows(s *Spec) {`
`93`	`93`	`}`
`94`	`94`	`}`
`95`	`95`
`96`		`-//nolint:unused // not used on all platforms`
	`96`	`+//nolint:nolintlint,unused // not used on all platforms`
`97`	`97`	`func setCPU(s *Spec) {`
`98`	`98`	`setResources(s)`
`99`	`99`	`if s.Linux != nil {`
`@@ -103,7 +103,7 @@ func setCPU(s *Spec) {`
`103`	`103`	`}`
`104`	`104`	`}`
`105`	`105`
`106`		`-//nolint:deadcode,unused // not used on all platforms`
	`106`	`+//nolint:deadcode,nolintlint,unused // not used on all platforms`
`107`	`107`	`func setCPUWindows(s *Spec) {`
`108`	`108`	`setResourcesWindows(s)`
`109`	`109`	`if s.Windows != nil {`
Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ func (c criService) ContainerStats(ctx context.Context, in runtime.ContainerSt`
`41`	`41`	`}`
`42`	`42`
`43`	`43`	`cs, err := c.containerMetrics(cntr.Metadata, resp.Metrics[0])`
`44`		`- if err != nil { //nolint:staticcheck // Ignore SA4023 as some platforms always return nil (stats unimplemented)`
	`44`	`+ if err != nil {`
`45`	`45`	`return nil, fmt.Errorf("failed to decode container metrics: %w", err)`
`46`	`46`	`}`
`47`	`47`	`return &runtime.ContainerStatsResponse{Stats: cs}, nil`
Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,7 @@ func (c *criService) toCRIContainerStats(`
`58`	`58`	`containerStats := new(runtime.ListContainerStatsResponse)`
`59`	`59`	`for _, cntr := range containers {`
`60`	`60`	`cs, err := c.containerMetrics(cntr.Metadata, statsMap[cntr.ID])`
`61`		`- if err != nil { //nolint:staticcheck // Ignore SA4023 as some platforms always return nil (metrics unimplemented)`
	`61`	`+ if err != nil {`
`62`	`62`	`return nil, fmt.Errorf("failed to decode container metrics for %q: %w", cntr.ID, err)`
`63`	`63`	`}`
`64`	`64`	`containerStats.Stats = append(containerStats.Stats, cs)`
Original file line number	Diff line number	Diff line change
`@@ -34,12 +34,12 @@ func (c *criService) PodSandboxStats(`
`34`	`34`	`}`
`35`	`35`
`36`	`36`	`metrics, err := metricsForSandbox(sandbox)`
`37`		`- if err != nil { //nolint:staticcheck // Ignore SA4023 as some platforms always return nil (unimplemented metrics)`
	`37`	`+ if err != nil {`
`38`	`38`	`return nil, fmt.Errorf("failed getting metrics for sandbox %s: %w", r.GetPodSandboxId(), err)`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`podSandboxStats, err := c.podSandboxStats(ctx, sandbox, metrics)`
`42`		`- if err != nil { //nolint:staticcheck // Ignore SA4023 as some platforms always return nil (unimplemented metrics)`
	`42`	`+ if err != nil {`
`43`	`43`	`return nil, fmt.Errorf("failed to decode pod sandbox metrics %s: %w", r.GetPodSandboxId(), err)`
`44`	`44`	`}`
`45`	`45`