Merge pull request #23614 from Luap99/lint-1.60.1

update golangci-lint to 1.60.1

Makefile

@@ -62,7 +62,7 @@ BUILDTAGS += ${EXTRA_BUILDTAGS}
 # N/B: This value is managed by Renovate, manual changes are
 # possible, as long as they don't disturb the formatting
 # (i.e. DO NOT ADD A 'v' prefix!)
-GOLANGCI_LINT_VERSION := 1.59.1
+GOLANGCI_LINT_VERSION := 1.60.1
 PYTHON ?= $(shell command -v python3 python|head -n1)
 PKG_MANAGER ?= $(shell command -v dnf yum|head -n1)
 # ~/.local/bin is not in PATH on all systems

cmd/podman-testing/create.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package main
 
 import (

cmd/podman-testing/data.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package main
 
 import (

cmd/podman-testing/layer.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package main
 
 import (

cmd/podman-testing/main.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package main
 
 import (

cmd/podman-testing/remove.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package main
 
 import (

cmd/podman-testing/store_supported.go

@@ -1,5 +1,4 @@
 //go:build linux && !remote
-// +build linux,!remote
 
 package main
 

cmd/podman/containers/commit.go

@@ -9,7 +9,6 @@ import (
 	"github.com/containers/common/pkg/completion"
 	"github.com/containers/podman/v5/cmd/podman/common"
 	"github.com/containers/podman/v5/cmd/podman/registry"
-	"github.com/containers/podman/v5/pkg/api/handlers"
 	"github.com/containers/podman/v5/pkg/domain/entities"
 	"github.com/spf13/cobra"
 )
@@ -105,9 +104,6 @@ func commit(cmd *cobra.Command, args []string) error {
 	if !commitOptions.Quiet {
 		commitOptions.Writer = os.Stderr
 	}
-	if len(commitOptions.Changes) > 0 {
-		commitOptions.Changes = handlers.DecodeChanges(commitOptions.Changes)
-	}
 	if len(configFile) > 0 {
 		cfg, err := os.ReadFile(configFile)
 		if err != nil {

cmd/podman/containers/create.go

@@ -314,7 +314,7 @@ func CreateInit(c *cobra.Command, vals entities.ContainerCreateOptions, isInfra
 		vals.ShmSizeSystemd = c.Flag("shm-size-systemd").Value.String()
 	}
 	if (c.Flag("dns").Changed || c.Flag("dns-option").Changed || c.Flag("dns-search").Changed) && vals.Net != nil && (vals.Net.Network.NSMode == specgen.NoNetwork || vals.Net.Network.IsContainer()) {
-		return vals, fmt.Errorf("conflicting options: dns and the network mode: " + string(vals.Net.Network.NSMode))
+		return vals, errors.New("conflicting options: dns and the network mode: " + string(vals.Net.Network.NSMode))
 	}
 	noHosts, err := c.Flags().GetBool("no-hosts")
 	if err != nil {

hack/golangci-lint.sh

@@ -18,11 +18,11 @@ BUILD_TAGS_TUNNEL="$BUILD_TAGS_DEFAULT,remote"
 BUILD_TAGS_REMOTE="remote,containers_image_openpgp"
 
 SKIP_DIRS_ABI=""
-SKIP_DIRS_TUNNEL="pkg/api,pkg/domain/infra/abi,internal/domain/infra/abi"
-SKIP_DIRS_REMOTE="libpod/events,pkg/api,pkg/domain/infra/abi,internal/domain/infra/abi,pkg/machine/qemu,pkg/trust,test"
+SKIP_DIRS_TUNNEL=""
+SKIP_DIRS_REMOTE="libpod/events,pkg/machine/qemu,pkg/machine/wsl,test"
 
 declare -a to_lint
-to_lint=(ABI TUNNEL REMOTE)
+to_lint=(ABI TUNNEL)
 
 # Special-case, for Darwin and Windows only "remote" linting is possible and required.
 if [[ "$GOOS" == "windows" ]] || [[ "$GOOS" == "darwin" ]]; then

internal/domain/entities/engine_testing.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package entities
 
 import (

internal/domain/entities/testing.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package entities
 
 type CreateStorageLayerOptions struct {

internal/domain/infra/abi/testing.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package abi
 
 import (

internal/domain/infra/abi/testing_test.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package abi
 
 import "github.com/containers/podman/v5/internal/domain/entities"

internal/domain/infra/tunnel/testing.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package tunnel
 
 import (

internal/domain/infra/tunnel/testing_test.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package tunnel
 
 import "github.com/containers/podman/v5/internal/domain/entities"

libpod/container_top_linux.go

@@ -411,7 +411,7 @@ func (c *Container) execPSinContainer(args []string) ([]string, error) {
 	if logrus.GetLevel() >= logrus.DebugLevel {
 		// If we're running in debug mode or higher, we might want to have a
 		// look at stderr which includes debug logs from conmon.
-		logrus.Debugf(errBuf.String())
+		logrus.Debug(errBuf.String())
 	}
 
 	if err := <-outErrChan; err != nil {

libpod/events/logfile_test.go

@@ -1,3 +1,5 @@
+//go:build linux || freebsd
+
 package events
 
 import (

pkg/annotations/validate.go

@@ -1,6 +1,7 @@
 package annotations
 
 import (
+	"errors"
 	"fmt"
 	"regexp"
 	"strings"
@@ -41,7 +42,7 @@ func isDNS1123Subdomain(value string) error {
 	}
 
 	if !dns1123SubdomainRegexp.MatchString(value) {
-		return fmt.Errorf(regexErrorMsg(dns1123SubdomainErrorMsg, dns1123SubdomainFmt, "example.com"))
+		return errors.New(regexErrorMsg(dns1123SubdomainErrorMsg, dns1123SubdomainFmt, "example.com"))
 	}
 
 	return nil

pkg/api/handlers/compat/auth.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/changes.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/containers.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/containers_archive.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/containers_attach.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/containers_create.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/containers_export.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/containers_logs.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/containers_pause.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/containers_prune.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/containers_restart.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/containers_start.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/containers_stats_freebsd.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/containers_stats_linux.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/containers_stop.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/containers_top.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/containers_unpause.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/events.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/exec.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/images.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (
@@ -137,7 +139,7 @@ func CommitContainer(w http.ResponseWriter, r *http.Request) {
 	options.Author = query.Author
 	options.Pause = query.Pause
 	options.Squash = query.Squash
-	options.Changes = handlers.DecodeChanges(query.Changes)
+	options.Changes = util.DecodeChanges(query.Changes)
 	if r.Body != nil {
 		defer r.Body.Close()
 		if options.CommitOptions.OverrideConfig, err = abi.DecodeOverrideConfig(r.Body); err != nil {

pkg/api/handlers/compat/images_build.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/images_history.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/images_prune.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/images_push.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/images_remove.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/images_save.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/images_search.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/images_tag.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/info.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/networks.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/ping.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (

pkg/api/handlers/compat/resize.go

@@ -1,3 +1,5 @@
+//go:build !remote
+
 package compat
 
 import (