Merge pull request #2522 from umagnus/release-1.28-golint

[release-1.28] fix golint and disable staticcheck

.github/workflows/static.yaml

@@ -16,4 +16,4 @@ jobs:
               uses: golangci/golangci-lint-action@v3
               with:
                   version: v1.60
-                  args: --timeout 10m
+                  args: -E=gofmt,unused,ineffassign,revive,misspell,exportloopref,asciicheck,bodyclose,depguard,dogsled,durationcheck,errname,forbidigo -D=staticcheck --timeout=30m0s

pkg/azuredisk/azure_common_linux_test.go

@@ -24,7 +24,7 @@ import (
 )
 
 func TestRescanAllVolumes(t *testing.T) {
-	if runtime.GOOS == "darwin" { // nolint: staticcheck
+	if runtime.GOOS == "darwin" {
 		t.Skipf("skip test on GOOS=%s", runtime.GOOS)
 	}
 	err := rescanAllVolumes(azureutils.NewOSIOHandler())

pkg/azuredisk/azuredisk.go

@@ -24,7 +24,7 @@ import (
 	"strings"
 	"time"
 
-	"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2022-08-01/compute" // nolint: staticcheck
+	"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2022-08-01/compute"
 	"github.com/container-storage-interface/spec/lib/go/csi"
 
 	"google.golang.org/grpc/codes"
@@ -458,7 +458,7 @@ func (d *DriverCore) waitForSnapshotReady(ctx context.Context, subsID, resourceG
 		return nil
 	}
 
-	timeTick := time.Tick(intervel) // nolint: staticcheck
+	timeTick := time.Tick(intervel)
 	timeAfter := time.After(timeout)
 	for {
 		select {

pkg/azuredisk/azuredisk_test.go

@@ -25,7 +25,7 @@ import (
 	"testing"
 	"time"
 
-	"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2022-08-01/compute" // nolint: staticcheck
+	"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2022-08-01/compute"
 	"github.com/Azure/go-autorest/autorest/date"
 	"github.com/golang/mock/gomock"
 	"github.com/stretchr/testify/assert"

pkg/azuredisk/azuredisk_v1.go

@@ -21,6 +21,6 @@ package azuredisk
 
 // NewDriver Creates a NewCSIDriver object. Assumes vendor version is equal to driver version &
 // does not support optional driver plugin info manifest field. Refer to CSI spec for more details.
-func NewDriver(options *DriverOptions) CSIDriver { // nolint: staticcheck
+func NewDriver(options *DriverOptions) CSIDriver {
 	return newDriverV1(options)
 }

pkg/azuredisk/azuredisk_v1_test.go

@@ -23,7 +23,7 @@ import (
 	"context"
 	"testing"
 
-	"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2022-08-01/compute" // nolint: staticcheck
+	"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2022-08-01/compute"
 	"github.com/golang/mock/gomock"
 	"github.com/stretchr/testify/assert"
 	consts "sigs.k8s.io/azuredisk-csi-driver/pkg/azureconstants"

pkg/azuredisk/controllerserver.go

@@ -25,7 +25,7 @@ import (
 	"strings"
 	"time"
 
-	"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2022-08-01/compute" // nolint: staticcheck
+	"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2022-08-01/compute"
 	"github.com/container-storage-interface/spec/lib/go/csi"
 
 	"google.golang.org/grpc/codes"

pkg/azuredisk/controllerserver_test.go

@@ -22,7 +22,7 @@ import (
 	"reflect"
 	"testing"
 
-	"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2022-08-01/compute" // nolint: staticcheck
+	"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2022-08-01/compute"
 	"github.com/Azure/go-autorest/autorest/date"
 	"github.com/container-storage-interface/spec/lib/go/csi"
 	"github.com/golang/mock/gomock"

pkg/azuredisk/fake_azuredisk.go

@@ -21,7 +21,7 @@ import (
 	"testing"
 	"time"
 
-	"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2022-08-01/compute" // nolint: staticcheck
+	"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2022-08-01/compute"
 	"github.com/container-storage-interface/spec/lib/go/csi"
 	"github.com/golang/mock/gomock"
 	"github.com/stretchr/testify/assert"

pkg/azuredisk/nodeserver.go

@@ -612,7 +612,7 @@ func (d *Driver) getDevicePathWithLUN(lunStr string) (string, error) {
 	scsiHostRescan(d.ioHandler, d.mounter)
 
 	newDevicePath := ""
-	err = wait.PollUntilContextTimeout(context.Background(), 1*time.Second, 2*time.Minute, true, func(context.Context) (bool, error) {
+	err = wait.PollImmediate(1*time.Second, 2*time.Minute, func() (bool, error) {
 		var err error
 		if newDevicePath, err = findDiskByLun(int(lun), d.ioHandler, d.mounter); err != nil {
 			return false, fmt.Errorf("azureDisk - findDiskByLun(%v) failed with error(%s)", lun, err)

pkg/azuredisk/nodeserver_test.go

@@ -29,7 +29,7 @@ import (
 	"syscall"
 	"testing"
 
-	"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2022-08-01/compute" // nolint: staticcheck
+	"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2022-08-01/compute"
 	"github.com/container-storage-interface/spec/lib/go/csi"
 	"github.com/golang/mock/gomock"
 	"github.com/stretchr/testify/assert"

pkg/azurediskplugin/main.go

@@ -120,8 +120,8 @@ func handle() {
 		GetNodeIDFromIMDS:            *getNodeIDFromIMDS,
 		WaitForSnapshotReady:         *waitForSnapshotReady,
 	}
-	driver := azuredisk.NewDriver(&driverOptions) // nolint: staticcheck
-	if driver == nil {                            // nolint: staticcheck
+	driver := azuredisk.NewDriver(&driverOptions)
+	if driver == nil {
 		klog.Fatalln("Failed to initialize azuredisk CSI Driver")
 	}
 	testingMock := false

pkg/azureutils/azure_disk_utils.go

@@ -30,7 +30,7 @@ import (
 	"time"
 	"unicode"
 
-	"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2022-08-01/compute" // nolint: staticcheck
+	"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2022-08-01/compute"
 	"github.com/container-storage-interface/spec/lib/go/csi"
 	"github.com/pborman/uuid"
 	v1 "k8s.io/api/core/v1"

pkg/azureutils/azure_disk_utils_test.go

@@ -28,7 +28,7 @@ import (
 	"testing"
 	"time"
 
-	"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2022-08-01/compute" // nolint: staticcheck
+	"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2022-08-01/compute"
 	"github.com/container-storage-interface/spec/lib/go/csi"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"

pkg/azureutils/azure_snapshot_utils.go

@@ -21,7 +21,7 @@ import (
 	"strconv"
 	"strings"
 
-	"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2022-08-01/compute" // nolint: staticcheck
+	"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2022-08-01/compute"
 	"github.com/container-storage-interface/spec/lib/go/csi"
 	"google.golang.org/grpc/codes"
 	"google.golang.org/grpc/status"

pkg/os/disk/types.go

@@ -27,38 +27,38 @@ type StoragePropertyID uint32
 
 const (
 	StorageDeviceProperty                  StoragePropertyID = 0
-	StorageAdapterProperty                 StoragePropertyID = 1
-	StorageDeviceIDProperty                StoragePropertyID = 2
-	StorageDeviceUniqueIDProperty          StoragePropertyID = 3
-	StorageDeviceWriteCacheProperty        StoragePropertyID = 4
-	StorageMiniportProperty                StoragePropertyID = 5
-	StorageAccessAlignmentProperty         StoragePropertyID = 6
-	StorageDeviceSeekPenaltyProperty       StoragePropertyID = 7
-	StorageDeviceTrimProperty              StoragePropertyID = 8
-	StorageDeviceWriteAggregationProperty  StoragePropertyID = 9
-	StorageDeviceDeviceTelemetryProperty   StoragePropertyID = 10
-	StorageDeviceLBProvisioningProperty    StoragePropertyID = 11
-	StorageDevicePowerProperty             StoragePropertyID = 12
-	StorageDeviceCopyOffloadProperty       StoragePropertyID = 13
-	StorageDeviceResiliencyProperty        StoragePropertyID = 14
-	StorageDeviceMediumProductType         StoragePropertyID = 15
-	StorageAdapterRpmbProperty             StoragePropertyID = 16
-	StorageAdapterCryptoProperty           StoragePropertyID = 17
-	StorageDeviceIoCapabilityProperty      StoragePropertyID = 18
-	StorageAdapterProtocolSpecificProperty StoragePropertyID = 19
-	StorageDeviceProtocolSpecificProperty  StoragePropertyID = 20
-	StorageAdapterTemperatureProperty      StoragePropertyID = 21
-	StorageDeviceTemperatureProperty       StoragePropertyID = 22
-	StorageAdapterPhysicalTopologyProperty StoragePropertyID = 23
-	StorageDevicePhysicalTopologyProperty  StoragePropertyID = 24
-	StorageDeviceAttributesProperty        StoragePropertyID = 25
-	StorageDeviceManagementStatus          StoragePropertyID = 26
-	StorageAdapterSerialNumberProperty     StoragePropertyID = 27
-	StorageDeviceLocationProperty          StoragePropertyID = 28
-	StorageDeviceNumaProperty              StoragePropertyID = 29
-	StorageDeviceZonedDeviceProperty       StoragePropertyID = 30
-	StorageDeviceUnsafeShutdownCount       StoragePropertyID = 31
-	StorageDeviceEnduranceProperty         StoragePropertyID = 32
+	StorageAdapterProperty                                   = 1
+	StorageDeviceIDProperty                                  = 2
+	StorageDeviceUniqueIDProperty                            = 3
+	StorageDeviceWriteCacheProperty                          = 4
+	StorageMiniportProperty                                  = 5
+	StorageAccessAlignmentProperty                           = 6
+	StorageDeviceSeekPenaltyProperty                         = 7
+	StorageDeviceTrimProperty                                = 8
+	StorageDeviceWriteAggregationProperty                    = 9
+	StorageDeviceDeviceTelemetryProperty                     = 10
+	StorageDeviceLBProvisioningProperty                      = 11
+	StorageDevicePowerProperty                               = 12
+	StorageDeviceCopyOffloadProperty                         = 13
+	StorageDeviceResiliencyProperty                          = 14
+	StorageDeviceMediumProductType                           = 15
+	StorageAdapterRpmbProperty                               = 16
+	StorageAdapterCryptoProperty                             = 17
+	StorageDeviceIoCapabilityProperty                        = 18
+	StorageAdapterProtocolSpecificProperty                   = 19
+	StorageDeviceProtocolSpecificProperty                    = 20
+	StorageAdapterTemperatureProperty                        = 21
+	StorageDeviceTemperatureProperty                         = 22
+	StorageAdapterPhysicalTopologyProperty                   = 23
+	StorageDevicePhysicalTopologyProperty                    = 24
+	StorageDeviceAttributesProperty                          = 25
+	StorageDeviceManagementStatus                            = 26
+	StorageAdapterSerialNumberProperty                       = 27
+	StorageDeviceLocationProperty                            = 28
+	StorageDeviceNumaProperty                                = 29
+	StorageDeviceZonedDeviceProperty                         = 30
+	StorageDeviceUnsafeShutdownCount                         = 31
+	StorageDeviceEnduranceProperty                           = 32
 )
 
 type StorageQueryType uint32
@@ -89,31 +89,31 @@ type StorageIdentifierCodeSet uint32
 
 const (
 	StorageIDCodeSetReserved StorageIdentifierCodeSet = 0
-	StorageIDCodeSetBinary   StorageIdentifierCodeSet = 1
-	StorageIDCodeSetASCII    StorageIdentifierCodeSet = 2
-	StorageIDCodeSetUtf8     StorageIdentifierCodeSet = 3
+	StorageIDCodeSetBinary                            = 1
+	StorageIDCodeSetASCII                             = 2
+	StorageIDCodeSetUtf8                              = 3
 )
 
 type StorageIdentifierType uint32
 
 const (
 	StorageIDTypeVendorSpecific           StorageIdentifierType = 0
-	StorageIDTypeVendorID                 StorageIdentifierType = 1
-	StorageIDTypeEUI64                    StorageIdentifierType = 2
-	StorageIDTypeFCPHName                 StorageIdentifierType = 3
-	StorageIDTypePortRelative             StorageIdentifierType = 4
-	StorageIDTypeTargetPortGroup          StorageIdentifierType = 5
-	StorageIDTypeLogicalUnitGroup         StorageIdentifierType = 6
-	StorageIDTypeMD5LogicalUnitIdentifier StorageIdentifierType = 7
-	StorageIDTypeScsiNameString           StorageIdentifierType = 8
+	StorageIDTypeVendorID                                       = 1
+	StorageIDTypeEUI64                                          = 2
+	StorageIDTypeFCPHName                                       = 3
+	StorageIDTypePortRelative                                   = 4
+	StorageIDTypeTargetPortGroup                                = 5
+	StorageIDTypeLogicalUnitGroup                               = 6
+	StorageIDTypeMD5LogicalUnitIdentifier                       = 7
+	StorageIDTypeScsiNameString                                 = 8
 )
 
 type StorageAssociationType uint32
 
 const (
 	StorageIDAssocDevice StorageAssociationType = 0
-	StorageIDAssocPort   StorageAssociationType = 1
-	StorageIDAssocTarget StorageAssociationType = 2
+	StorageIDAssocPort                          = 1
+	StorageIDAssocTarget                        = 2
 )
 
 type StorageIdentifier struct {

pkg/util/util.go

@@ -124,13 +124,13 @@ func MakeFile(pathname string) error {
 }
 
 type VolumeLocks struct {
-	locks sets.Set[string]
+	locks sets.String
 	mux   sync.Mutex
 }
 
 func NewVolumeLocks() *VolumeLocks {
 	return &VolumeLocks{
-		locks: sets.New[string](),
+		locks: sets.NewString(),
 	}
 }
 

test/e2e/pre_provisioning_test.go

@@ -24,7 +24,6 @@ import (
 
 	"github.com/container-storage-interface/spec/lib/go/csi"
 	"github.com/onsi/ginkgo/v2"
-	"github.com/onsi/gomega"
 	v1 "k8s.io/api/core/v1"
 	clientset "k8s.io/client-go/kubernetes"
 	"k8s.io/kubernetes/test/e2e/framework"
@@ -172,7 +171,7 @@ var _ = ginkgo.Describe("Pre-Provisioned", func() {
 			req := makeCreateVolumeReq("invalid-maxShares", 256)
 			req.Parameters = map[string]string{"maxShares": "0"}
 			_, err := azurediskDriver.CreateVolume(ctx, req)
-			gomega.Expect(err).To(gomega.HaveOccurred())
+			framework.ExpectError(err)
 		})
 
 		ginkgo.It("should succeed when attaching a shared block volume to multiple pods [disk.csi.azure.com][shared disk]", func(ctx ginkgo.SpecContext) {

test/e2e/suite_test.go

@@ -28,6 +28,7 @@ import (
 	"testing"
 
 	"github.com/onsi/ginkgo/v2"
+	"github.com/onsi/ginkgo/v2/reporters"
 	"github.com/onsi/gomega"
 	"github.com/pborman/uuid"
 	"k8s.io/kubernetes/test/e2e/framework"
@@ -265,9 +266,8 @@ func TestE2E(t *testing.T) {
 	if reportDir == "" {
 		reportDir = defaultReportDir
 	}
-	_, reporterConfig := ginkgo.GinkgoConfiguration()
-	reporterConfig.JUnitReport = path.Join(reportDir, "junit_01.xml")
-	ginkgo.RunSpecs(t, "AzureDisk CSI Driver End-to-End Tests", reporterConfig)
+	r := []ginkgo.Reporter{reporters.NewJUnitReporter(path.Join(reportDir, "junit_01.xml"))}
+	ginkgo.RunSpecsWithDefaultAndCustomReporters(t, "AzureDisk CSI Driver End-to-End Tests", r)
 }
 
 func execTestCmd(cmds []testCmd) {

test/e2e/testsuites/dynamically_provisioned_azuredisk_detach_tester.go

@@ -21,7 +21,6 @@ import (
 	"fmt"
 	"time"
 
-	"github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/compute/armcompute/v5"
 	compute "github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/compute/armcompute/v5"
 	"github.com/onsi/ginkgo/v2"
 	"github.com/onsi/gomega"
@@ -87,17 +86,17 @@ func (t *DynamicallyProvisionedAzureDiskDetach) Run(ctx context.Context, client
 		framework.ExpectNoError(err, fmt.Sprintf("Error getting client for azuredisk %v", err))
 		disktest, err := disksClient.Get(ctx, resourceGroup, diskName)
 		framework.ExpectNoError(err, fmt.Sprintf("Error getting disk for azuredisk %v", err))
-		gomega.Expect(compute.DiskStateAttached).To(gomega.Equal(*disktest.Properties.DiskState))
+		framework.ExpectEqual(compute.DiskStateAttached, *disktest.Properties.DiskState)
 
 		ginkgo.By("begin to delete the pod")
 		tpod.Cleanup(ctx)
 
-		err = wait.PollUntilContextTimeout(ctx, 15*time.Second, 10*time.Minute, false, func(context.Context) (bool, error) {
+		err = wait.Poll(15*time.Second, 10*time.Minute, func() (bool, error) {
 			disktest, err := disksClient.Get(ctx, resourceGroup, diskName)
 			if err != nil {
 				return false, fmt.Errorf("Error getting disk for azuredisk %v", err)
 			}
-			if *disktest.Properties.DiskState == armcompute.DiskStateUnattached {
+			if *disktest.Properties.DiskState == compute.DiskStateUnattached {
 				return true, nil
 			}
 			ginkgo.By(fmt.Sprintf("current disk state(%v) is not in unattached state, wait and recheck", *disktest.Properties.DiskState))

test/e2e/testsuites/dynamically_provisioned_azuredisk_tag_tester.go

@@ -90,28 +90,28 @@ func (t *DynamicallyProvisionedAzureDiskWithTag) Run(ctx context.Context, client
 
 		for k, v := range test {
 			_, ok := disktest.Tags[k]
-			gomega.Expect(ok).To(gomega.Equal(true))
+			framework.ExpectEqual(ok, true)
 			if ok {
-				gomega.Expect(*disktest.Tags[k]).To(gomega.Equal(v))
+				framework.ExpectEqual(*disktest.Tags[k], v)
 			}
 		}
 		tag, ok := disktest.Tags["kubernetes.io-created-for-pv-name"]
-		gomega.Expect(ok).To(gomega.Equal(true))
-		gomega.Expect(tag != nil).To(gomega.Equal(true))
+		framework.ExpectEqual(ok, true)
+		framework.ExpectEqual(tag != nil, true)
 		if tag != nil {
 			ginkgo.By(fmt.Sprintf("kubernetes.io-created-for-pv-name: %s", *tag))
 		}
 
 		tag, ok = disktest.Tags["kubernetes.io-created-for-pvc-name"]
-		gomega.Expect(ok).To(gomega.Equal(true))
-		gomega.Expect(tag != nil).To(gomega.Equal(true))
+		framework.ExpectEqual(ok, true)
+		framework.ExpectEqual(tag != nil, true)
 		if tag != nil {
 			ginkgo.By(fmt.Sprintf("kubernetes.io-created-for-pvc-name: %s", *tag))
 		}
 
 		tag, ok = disktest.Tags["kubernetes.io-created-for-pvc-namespace"]
-		gomega.Expect(ok).To(gomega.Equal(true))
-		gomega.Expect(tag != nil).To(gomega.Equal(true))
+		framework.ExpectEqual(ok, true)
+		framework.ExpectEqual(tag != nil, true)
 		if tag != nil {
 			ginkgo.By(fmt.Sprintf("kubernetes.io-created-for-pvc-namespace: %s", *tag))
 		}

test/e2e/testsuites/dynamically_provisioned_resize_volume_tester.go

@@ -120,7 +120,7 @@ func (t *DynamicallyProvisionedResizeVolumeTest) Run(ctx context.Context, client
 
 	var newPv *v1.PersistentVolume
 	var newPvSize resource.Quantity
-	err = wait.PollUntilContextTimeout(ctx, 30*time.Second, 10*time.Minute, true, func(context.Context) (bool, error) {
+	err = wait.PollImmediate(30*time.Second, 10*time.Minute, func() (bool, error) {
 		//takes 3-6 minutes on average for dynamic resize
 		ginkgo.By("checking the resizing PV result")
 		newPv, _ = client.CoreV1().PersistentVolumes().Get(ctx, newPvc.Spec.VolumeName, metav1.GetOptions{})