cleanup: fix static checks

fix SA1019 static check to replace
io/utils with os package and sets
with generic sets

Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>

internal/kms/vault.go

@@ -19,7 +19,6 @@ package kms
 import (
 	"errors"
 	"fmt"
-	"io/ioutil"
 	"os"
 	"path/filepath"
 	"strconv"
@@ -254,7 +253,6 @@ func (vc *vaultConnection) initConnection(config map[string]interface{}) error {
 // initCertificates sets VAULT_* environment variables in the vc.vaultConfig map,
 // these settings will be used when connecting to the Vault service with
 // vc.connectVault().
-//
 func (vc *vaultConnection) initCertificates(config map[string]interface{}, secrets map[string]string) error {
 	vaultConfig := make(map[string]interface{})
 
@@ -481,9 +479,9 @@ func detectAuthMountPath(path string) (string, error) {
 }
 
 // createTempFile writes data to a temporary file that contains the pattern in
-// the filename (see ioutil.TempFile for details).
+// the filename (see os.CreateTemp for details).
 func createTempFile(pattern string, data []byte) (string, error) {
-	t, err := ioutil.TempFile("", pattern)
+	t, err := os.CreateTemp("", pattern)
 	if err != nil {
 		return "", fmt.Errorf("failed to create temporary file: %w", err)
 	}

internal/kms/vault_sa.go

@@ -20,7 +20,6 @@ import (
 	"context"
 	"errors"
 	"fmt"
-	"io/ioutil"
 	"os"
 
 	"github.com/libopenstorage/secrets/vault"
@@ -321,7 +320,7 @@ func (kms *vaultTenantSA) getToken() (string, error) {
 // linked from the ServiceAccount. This path can then be used in place of the
 // standard `/var/run/secrets/kubernetes.io/serviceaccount/token` location.
 func (kms *vaultTenantSA) getTokenPath() (string, error) {
-	dir, err := ioutil.TempDir("", kms.tenantSAName)
+	dir, err := os.MkdirTemp("", kms.tenantSAName)
 	if err != nil {
 		return "", fmt.Errorf("failed to create directory for ServiceAccount %s/%s: %w", kms.tenantSAName, kms.Tenant, err)
 	}

internal/rbd/rbd_util_test.go

@@ -19,7 +19,6 @@ package rbd
 import (
 	"context"
 	"errors"
-	"io/ioutil"
 	"os"
 	"strings"
 	"testing"
@@ -252,7 +251,7 @@ func TestStrategicActionOnLogFile(t *testing.T) {
 
 	var logFile [3]string
 	for i := 0; i < 3; i++ {
-		f, err := ioutil.TempFile(tmpDir, "rbd-*.log")
+		f, err := os.CreateTemp(tmpDir, "rbd-*.log")
 		if err != nil {
 			t.Errorf("creating tempfile failed: %v", err)
 		}

internal/util/credentials.go

@@ -19,7 +19,6 @@ package util
 import (
 	"errors"
 	"fmt"
-	"io/ioutil"
 	"os"
 )
 
@@ -43,7 +42,7 @@ type Credentials struct {
 }
 
 func storeKey(key string) (string, error) {
-	tmpfile, err := ioutil.TempFile(tmpKeyFileLocation, tmpKeyFileNamePrefix)
+	tmpfile, err := os.CreateTemp(tmpKeyFileLocation, tmpKeyFileNamePrefix)
 	if err != nil {
 		return "", fmt.Errorf("error creating a temporary keyfile: %w", err)
 	}

internal/util/idlocker.go

@@ -33,14 +33,14 @@ const (
 // VolumeLocks implements a map with atomic operations. It stores a set of all volume IDs
 // with an ongoing operation.
 type VolumeLocks struct {
-	locks sets.String
+	locks sets.Set[string]
 	mux   sync.Mutex
 }
 
 // NewVolumeLocks returns new VolumeLocks.
 func NewVolumeLocks() *VolumeLocks {
 	return &VolumeLocks{
-		locks: sets.NewString(),
+		locks: sets.New[string](),
 	}
 }
 

internal/util/log/log_utils_test.go

@@ -19,7 +19,6 @@ package log
 import (
 	"errors"
 	"fmt"
-	"io/ioutil"
 	"os"
 	"strings"
 	"testing"
@@ -28,7 +27,7 @@ import (
 func TestGzipLogFile(t *testing.T) {
 	t.Parallel()
 	tmpDir := t.TempDir()
-	logFile, err := ioutil.TempFile(tmpDir, "rbd-*.log")
+	logFile, err := os.CreateTemp(tmpDir, "rbd-*.log")
 	if err != nil {
 		fmt.Println(err)
 	}