From 7a0efd326b7596926b543c87d2384df15be3a615 Mon Sep 17 00:00:00 2001
From: qingsenLi <li.qingsen5@zte.com.cn>
Date: Sat, 15 Jun 2019 10:53:50 +0800
Subject: [PATCH] fix golint failures for pkg/volume/scaleio and
 pkg/volume/storageos

---
 hack/.golint_failures                  |  2 --
 pkg/volume/scaleio/sio_util.go         | 25 ++++++++++++-------------
 pkg/volume/scaleio/sio_util_test.go    |  2 +-
 pkg/volume/storageos/storageos_util.go |  7 +++++--
 4 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/hack/.golint_failures b/hack/.golint_failures
index 8469ff3f03f..bb3654f0f16 100644
--- a/hack/.golint_failures
+++ b/hack/.golint_failures
@@ -307,8 +307,6 @@ pkg/volume/csi/fake
 pkg/volume/git_repo
 pkg/volume/iscsi
 pkg/volume/rbd
-pkg/volume/scaleio
-pkg/volume/storageos
 pkg/volume/testing
 pkg/volume/util/fs
 pkg/volume/util/volumepathhandler
diff --git a/pkg/volume/scaleio/sio_util.go b/pkg/volume/scaleio/sio_util.go
index e1b5116318f..98872e717ba 100644
--- a/pkg/volume/scaleio/sio_util.go
+++ b/pkg/volume/scaleio/sio_util.go
@@ -76,13 +76,12 @@ var (
 	sdcGUIDLabelName = "scaleio.sdcGUID"
 	sdcRootPath      = "/opt/emc/scaleio/sdc/bin"
 
-	secretNotFoundErr              = errors.New("secret not found")
-	configMapNotFoundErr           = errors.New("configMap not found")
-	gatewayNotProvidedErr          = errors.New("ScaleIO gateway not provided")
-	secretRefNotProvidedErr        = errors.New("secret ref not provided")
-	systemNotProvidedErr           = errors.New("ScaleIO system not provided")
-	storagePoolNotProvidedErr      = errors.New("ScaleIO storage pool not provided")
-	protectionDomainNotProvidedErr = errors.New("ScaleIO protection domain not provided")
+	errSecretNotFound              = errors.New("secret not found")
+	errGatewayNotProvided          = errors.New("ScaleIO gateway not provided")
+	errSecretRefNotProvided        = errors.New("secret ref not provided")
+	errSystemNotProvided           = errors.New("ScaleIO system not provided")
+	errStoragePoolNotProvided      = errors.New("ScaleIO storage pool not provided")
+	errProtectionDomainNotProvided = errors.New("ScaleIO protection domain not provided")
 )
 
 // mapVolumeSpec maps attributes from either ScaleIOVolumeSource  or ScaleIOPersistentVolumeSource to config
@@ -118,19 +117,19 @@ func mapVolumeSpec(config map[string]string, spec *volume.Spec) {
 
 func validateConfigs(config map[string]string) error {
 	if config[confKey.gateway] == "" {
-		return gatewayNotProvidedErr
+		return errGatewayNotProvided
 	}
 	if config[confKey.secretName] == "" {
-		return secretRefNotProvidedErr
+		return errSecretRefNotProvided
 	}
 	if config[confKey.system] == "" {
-		return systemNotProvidedErr
+		return errSystemNotProvided
 	}
 	if config[confKey.storagePool] == "" {
-		return storagePoolNotProvidedErr
+		return errStoragePoolNotProvided
 	}
 	if config[confKey.protectionDomain] == "" {
-		return protectionDomainNotProvidedErr
+		return errProtectionDomainNotProvided
 	}
 
 	return nil
@@ -222,7 +221,7 @@ func attachSecret(plug *sioPlugin, namespace string, configData map[string]strin
 	secretMap, err := volutil.GetSecretForPV(namespace, secretRefName, sioPluginName, kubeClient)
 	if err != nil {
 		klog.Error(log("failed to get secret: %v", err))
-		return secretNotFoundErr
+		return errSecretNotFound
 	}
 	// merge secret data
 	for key, val := range secretMap {
diff --git a/pkg/volume/scaleio/sio_util_test.go b/pkg/volume/scaleio/sio_util_test.go
index a574a46af5e..758b3710766 100644
--- a/pkg/volume/scaleio/sio_util_test.go
+++ b/pkg/volume/scaleio/sio_util_test.go
@@ -93,7 +93,7 @@ func TestUtilValidateConfigs(t *testing.T) {
 		confKey.secretName: "sio-secret",
 		confKey.system:     "sio",
 	}
-	if err := validateConfigs(data); err != gatewayNotProvidedErr {
+	if err := validateConfigs(data); err != errGatewayNotProvided {
 		t.Error("Expecting error for missing gateway, but did not get it")
 	}
 }
diff --git a/pkg/volume/storageos/storageos_util.go b/pkg/volume/storageos/storageos_util.go
index 8a1a489b913..c7f430ea5d4 100644
--- a/pkg/volume/storageos/storageos_util.go
+++ b/pkg/volume/storageos/storageos_util.go
@@ -37,9 +37,12 @@ const (
 	modeFile
 	modeUnsupported
 
-	ErrDeviceNotFound     = "device not found"
+	//ErrDeviceNotFound defines "device not found"
+	ErrDeviceNotFound = "device not found"
+	//ErrDeviceNotSupported defines "device not supported"
 	ErrDeviceNotSupported = "device not supported"
-	ErrNotAvailable       = "not available"
+	//ErrNotAvailable defines "not available"
+	ErrNotAvailable = "not available"
 )
 
 type deviceType int